[webkit-reviews] review granted: [Bug 14411] Regression: WebNetscapePluginPackage overagressively sets CurApRefNum, causes BBEdit to malfunction : [Attachment 15246] Proposed patch for bug.

[bugzilla-daemon at webkit.org]

Darin Adler <darin at apple.com> has granted Mark Rowe (bdash)
<bdash at webkit.org>'s request for review:
Bug 14411: Regression: WebNetscapePluginPackage overagressively sets
CurApRefNum, causes BBEdit to malfunction
http://bugs.webkit.org/show_bug.cgi?id=14411

Attachment 15246: Proposed patch for bug.
http://bugs.webkit.org/attachment.cgi?id=15246&action=edit

------- Additional Comments from Darin Adler <darin at apple.com>
Good change.

Tab in the ChangeLog will need ot be removed to check in. Also I'd like the
comment to mention the name and brief description of the bug, not just the ID.

Also, the (void) cast, while a common way to indicate that a function value is
ignored, is not something we generally do in WebKit/Core code. I'd prefer to
leave it out unless we have a tool that complains about ignored return values.
I think the emphasis on the fact that the function is being used for its side
effect is a good thing, but I think the comment already makes that clear and
the ugly syntax is only really worthwhile in code where it's consistently used.