Summary: | Use more Element iterators in SVG code. | ||||||||
---|---|---|---|---|---|---|---|---|---|
Product: | WebKit | Reporter: | Andreas Kling <kling> | ||||||
Component: | SVG | Assignee: | Andreas Kling <kling> | ||||||
Status: | RESOLVED FIXED | ||||||||
Severity: | Normal | CC: | commit-queue, d-r, fmalita, gyuyoung.kim, pdr, schenney, zimmermann | ||||||
Priority: | P2 | ||||||||
Version: | 528+ (Nightly build) | ||||||||
Hardware: | Unspecified | ||||||||
OS: | Unspecified | ||||||||
Attachments: |
|
Description
Andreas Kling
2013-09-10 15:09:28 PDT
Created attachment 211251 [details]
Patch
Comment on attachment 211251 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=211251&action=review > Source/WebCore/svg/SVGUseElement.cpp:615 > + for (auto it = childrenOfType<SVGElement>(target).begin(), end = childrenOfType<SVGElement>(target).end(); it != end; ++it) { I like doing auto svgChildren = childrenOfType<SVGElement>(this); for (auto it = svgChildren.begin(), end = svgChildren.end(); it != end; ++it) Keeps the for statement shorter and avoids repetition. Created attachment 213873 [details]
Lander
Comment on attachment 213873 [details] Lander Clearing flags on attachment: 213873 Committed r157216: <http://trac.webkit.org/changeset/157216> All reviewed patches have been landed. Closing bug. |