[webkit-reviews] review denied: [Bug 35607] Allow building smoothly on win32 and win64 using GCC : [Attachment 53630] WebCore patch for mingw-w64
bugzilla-daemon at webkit.org
bugzilla-daemon at webkit.org
Mon Apr 26 06:33:18 PDT 2010
Adam Roben (aroben) <aroben at apple.com> has denied vanboxem.ruben at gmail.com's
request for review:
Bug 35607: Allow building smoothly on win32 and win64 using GCC
https://bugs.webkit.org/show_bug.cgi?id=35607
Attachment 53630: WebCore patch for mingw-w64
https://bugs.webkit.org/attachment.cgi?id=53630&action=review
------- Additional Comments from Adam Roben (aroben) <aroben at apple.com>
> + * WebCore/bridge/npapi.h: lparam and wparam are 64-bit on WIN64,
uintptr_t keeps 32-bit compatibility
You should mention that this change matches Firefox. You could even reference
the bug you filed.
> + * WebCore/platform/Arena.h: fix pointer cast on win64
This comment is a little strange. You changed a typedef, but the comment talks
about a cast. I think you should clear this up.
It might also be worth mentioning that unsigned long is 64 bits on 64-bit Mac
builds, but only 32 bits on 64-bit Windows builds, while uintptr_t is 64 bits
on both.
Let's get these ChangeLog comments cleared up and then I'll r+ the patch.
Thanks for sticking with it!
More information about the webkit-reviews
mailing list