[webkit-reviews] review requested: [Bug 34953] Implement DEFINE_STUB_FUNCTION for WinCE : [Attachment 60889] Patch (current trunk)
bugzilla-daemon at webkit.org
bugzilla-daemon at webkit.org
Thu Jul 8 08:41:29 PDT 2010
Patrick R. Gansterer <paroga at paroga.com> has asked for review:
Bug 34953: Implement DEFINE_STUB_FUNCTION for WinCE
https://bugs.webkit.org/show_bug.cgi?id=34953
Attachment 60889: Patch (current trunk)
https://bugs.webkit.org/attachment.cgi?id=60889&action=review
------- Additional Comments from Patrick R. Gansterer <paroga at paroga.com>
(In reply to comment #15)
> Why the JITSTUBS_OFFSET is hard-coded in the pro file? Could not we use a
better approach? like parsing it from JITStups.cpp or Platform.h with the same
script?
It depends on USE(JSVALUE32_64) and and to get the correct one we need to pass
it to a preprecessor with all the defines.
I don't like it there too. Maybe you have a good idea to get rid of it?
More information about the webkit-reviews
mailing list