[webkit-reviews] review granted: [Bug 121082] Use unique_ptr instead of deleteAllValues in XPath : [Attachment 213753] Patch

bugzilla-daemon at webkit.org bugzilla-daemon at webkit.org
Wed Oct 9 11:06:49 PDT 2013


Anders Carlsson <andersca at apple.com> has granted Darin Adler
<darin at apple.com>'s request for review:
Bug 121082: Use unique_ptr instead of deleteAllValues in XPath
https://bugs.webkit.org/show_bug.cgi?id=121082

Attachment 213753: Patch
https://bugs.webkit.org/attachment.cgi?id=213753&action=review

------- Additional Comments from Anders Carlsson <andersca at apple.com>
View in context: https://bugs.webkit.org/attachment.cgi?id=213753&action=review


> Source/WebCore/xml/XPathExpression.h:53
> +	   XPathExpression(std::unique_ptr<XPath::Expression>);

Could make this explicit for good measure.

> Source/WebCore/xml/XPathNodeSet.h:39
> +	       NodeSet(PassRefPtr<Node> node) : m_isSorted(true),
m_subtreesAreDisjoint(false), m_nodes(1, node) { }

Explicit (if possible).


More information about the webkit-reviews mailing list