[webkit-reviews] review granted: [Bug 50316] Split DECLARE_ANIMATED_PROPERTY* in DECLARE/DEFINE parts for SVGLength : [Attachment 75262] Patch

[bugzilla-daemon at webkit.org]

Xan Lopez <xan.lopez at gmail.com> has granted Nikolas Zimmermann
<zimmermann at kde.org>'s request for review:
Bug 50316: Split DECLARE_ANIMATED_PROPERTY* in DECLARE/DEFINE parts for
SVGLength
https://bugs.webkit.org/show_bug.cgi?id=50316

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

------- Additional Comments from Xan Lopez <xan.lopez at gmail.com>
I think this is one good example of a patch where mixing style fixes makes
things harder to read :)

Looks reasonable to me otherwise. You might want to get rid of the file list in
the ChangeLog if you are not going to use it to add inline comments, since it's
pretty big.