[webkit-reviews] review granted: [Bug 51981] [Qt] [Symbian] Fix building NPAPI support : [Attachment 78096] proposed patch

bugzilla-daemon at webkit.org bugzilla-daemon at webkit.org
Mon Jan 17 00:07:39 PST 2011


Simon Hausmann <hausmann at webkit.org> has granted Laszlo Gombos
<laszlo.1.gombos at nokia.com>'s request for review:
Bug 51981: [Qt] [Symbian] Fix building NPAPI support
https://bugs.webkit.org/show_bug.cgi?id=51981

Attachment 78096: proposed patch
https://bugs.webkit.org/attachment.cgi?id=78096&action=review

------- Additional Comments from Simon Hausmann <hausmann at webkit.org>
I think this change is correct. A header file should not rely on extern "C"
being set but declare it itself. Plus npapi.h already has the extern "C"
declarations a bit further down the file.


More information about the webkit-reviews mailing list