Summary: | Uninitialized member m_createdByParser in WebCore::ProcessingInstruction | ||||||||
---|---|---|---|---|---|---|---|---|---|
Product: | WebKit | Reporter: | Dan Kegel <dank> | ||||||
Component: | DOM | Assignee: | Nobody <webkit-unassigned> | ||||||
Status: | RESOLVED FIXED | ||||||||
Severity: | Normal | CC: | cdumez, commit-queue | ||||||
Priority: | P2 | ||||||||
Version: | 528+ (Nightly build) | ||||||||
Hardware: | All | ||||||||
OS: | All | ||||||||
Attachments: |
|
Description
Dan Kegel
2009-11-03 15:19:18 PST
The fix looks right to me. Would you be willing to submit it for review, as described in <http://webkit.org/coding/contributing.html>? Sure, I'll do that in the morning, thanks. Created attachment 42484 [details]
Add mising initializer
Here y'go!
Comment on attachment 42484 [details]
Add mising initializer
You can leave the "Reviewed by NOBODY (OOPS!)." line alone and our scripts will fill it in automatically with Alexey's full name when landing. It would be better for you to paste an un-altered NOBODY line, or to correct that to say "Alexey Proskuryakov." (with a period), before we land this.
LGTM, and someone could land this manually with that fix, but we should get the ChangeLog completely right (either left as NOBODY (OOPS!) or changed to Alexey's full name with a period) if we want the commit-queue to land this.
Thanks for the patch!
Created attachment 42511 [details]
2nd draft, with right 'reviewed by'
In case it's useful, here is the patch with the suggested change.
Comment on attachment 42511 [details]
2nd draft, with right 'reviewed by'
r+, cq+, commit bot to substitute the right reviewer.
Comment on attachment 42511 [details] 2nd draft, with right 'reviewed by' Clearing flags on attachment: 42511 Committed r50571: <http://trac.webkit.org/changeset/50571> All reviewed patches have been landed. Closing bug. Mass moving XML DOM bugs to the "DOM" Component. |