[Webkit-unassigned] [Bug 3749] XHTML 1.1 Ruby Annotation

[bugzilla-daemon at webkit.org]

https://bugs.webkit.org/show_bug.cgi?id=3749





------- Comment #12 from rolandsteiner at google.com  2009-06-28 20:44 PDT -------
(In reply to comment #10 and comment #11)

Will try to break up the patch into smaller chunks, as suggested here and on
the mailing list.

> A few unsolicited code reviews for some style issues:
[snipped]

Thanks! Will correct those places.

> - For RenderRuby*::isChildAllowed, WebKit doesn't use line breaks around || , I
> think.

FWIW, I saw both styles in the code (IMHO, line breaks make it easier to
add/remove entries), but as the code has solidified, I can remove them.


-- 
Configure bugmail: https://bugs.webkit.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug, or are watching the assignee.