[webkit-reviews] review granted: [Bug 6107] Decoder should use Shared<T> and clients should use RefPtr<Decoder> : [Attachment 5107] Proposed patch.

bugzilla-request-daemon at opendarwin.org bugzilla-request-daemon at opendarwin.org
Fri Dec 16 07:11:08 PST 2005


Darin Adler <darin at apple.com> has granted Eric Seidel
<macdome at opendarwin.org>'s request for review:
Bug 6107: Decoder should use Shared<T> and clients should use RefPtr<Decoder>
http://bugzilla.opendarwin.org/show_bug.cgi?id=6107

Attachment 5107: Proposed patch.
http://bugzilla.opendarwin.org/attachment.cgi?id=5107&action=edit

------- Additional Comments from Darin Adler <darin at apple.com>
Looks fine, although I don't think ~DocumentImpl needs to explicitly set
m_decoder to 0 any more.

r=me



More information about the webkit-reviews mailing list