[webkit-reviews] review granted: [Bug 10745] SVGLength object needs some weight loss : [Attachment 22318] Updated patch

[bugzilla-daemon at webkit.org]

Eric Seidel <eric at webkit.org> has granted Nikolas Zimmermann
<zimmermann at kde.org>'s request for review:
Bug 10745: SVGLength object needs some weight loss
https://bugs.webkit.org/show_bug.cgi?id=10745

Attachment 22318: Updated patch
https://bugs.webkit.org/attachment.cgi?id=22318&action=edit

------- Additional Comments from Eric Seidel <eric at webkit.org>
Looks good.

Unfortunately if these are allocated on the heap this won't actually save
memory (due to 16 byte alignment), however if SVGLength is ever used as a
member, the owning object will shrink by 4 bytes. :)

Looks good though.

Could you please explain the results of:

$getterContentHead = "IMPL->value(0 /* FIXME */";

Perhaps you could even add a failing test for these two /* FIXME */ cases?