ASSERT will be defined in both conditional branches of the code following the proposed patch. The explicit undef avoids potential warning messages during compilation.
Created attachment 32412 [details] Code patch for Assertion.h 2nd attempt to add attachment - failed first time
I don't understand the reason for this patch?
Created attachment 32780 [details] Second try. Changed the patch so that it only impacts the SYMBIAN platform and reflect the intention of this change better. In addition fixed the ChangeLog (filling in for Norbert.) The reason for this patch is to kill the following compiler warning: "\WebKit\JavaScriptCore\wtf\Assertions.h", line 146: Warning: #47-D: incompatible redefinition of macro "ASSERT" (declared at line 3029 of "\epoc32\include\e32def.h") #define ASSERT(assertion) ((void)0)
(In reply to comment #3) > Created an attachment (id=32780) [details] > Second try. > > Changed the patch so that it only impacts the SYMBIAN platform and reflect the > intention of this change better. In addition fixed the ChangeLog (filling in > for Norbert.) > > The reason for this patch is to kill the following compiler warning: > > "\WebKit\JavaScriptCore\wtf\Assertions.h", line 146: Warning: #47-D: > incompatible redefinition of macro "ASSERT" (declared at line 3029 of > "\epoc32\include\e32def.h") > #define ASSERT(assertion) ((void)0) The second patch should obsolete the first patch from Norbert - I do not seems to have rights to make the first patch obsolete.
Comment on attachment 32412 [details] Code patch for Assertion.h Obsolete according to lazlos.
Comment on attachment 32780 [details] Second try. r=me. I'll mention Symbian in the changelog when landing, to make it a bit clearer.
Landed in r45907