Summary: | Small improvements to RenderBox/LayoutUnit casting in MathML | ||||||
---|---|---|---|---|---|---|---|
Product: | WebKit | Reporter: | Frédéric Wang (:fredw) <fred.wang> | ||||
Component: | MathML | Assignee: | Frédéric Wang (:fredw) <fred.wang> | ||||
Status: | RESOLVED FIXED | ||||||
Severity: | Normal | CC: | commit-queue, dbarton, esprehn+autocc, glenn, kondapallykalyan, mrobinson, svillar | ||||
Priority: | P2 | ||||||
Version: | WebKit Nightly Build | ||||||
Hardware: | All | ||||||
OS: | All | ||||||
Bug Depends on: | 153208, 153742, 153917 | ||||||
Bug Blocks: | |||||||
Attachments: |
|
Description
Frédéric Wang (:fredw)
2016-05-20 06:16:18 PDT
Created attachment 279480 [details]
Patch
Comment on attachment 279480 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=279480&action=review > Source/WebCore/rendering/mathml/RenderMathMLUnderOver.cpp:58 > + RenderBox* base = firstChildBox(); Typically in new code, we would use auto* here instead of writing RenderBox*. > Source/WebCore/rendering/mathml/RenderMathMLUnderOver.cpp:78 > + for (RenderBox* child = firstChildBox(); child; child = child->nextSiblingBox()) { Ditto. |