[webkit-reviews] review granted: [Bug 39194] RTL fixes for multicol layout : [Attachment 56207] Patch

bugzilla-daemon at webkit.org bugzilla-daemon at webkit.org
Sun May 16 20:15:14 PDT 2010


mitz at webkit.org has granted Dave Hyatt <hyatt at apple.com>'s request for review:
Bug 39194: RTL fixes for multicol layout
https://bugs.webkit.org/show_bug.cgi?id=39194

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

------- Additional Comments from mitz at webkit.org

> +    if (colCount == 0)
> +	   return;

Should be if (!colCount)

> +    if (colCount == 0)
> +	   return false;

Should be if (!colCount)

> +    if (colCount == 0)
> +	   return;

Ditto.


More information about the webkit-reviews mailing list