[Webkit-unassigned] [Bug 49353] ComplexTextController not prepared to handle multiple runs

[bugzilla-daemon at webkit.org]

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


mitz at webkit.org changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
  Attachment #73576|review?                     |review+, commit-queue-
               Flag|                            |




--- Comment #5 from mitz at webkit.org  2010-11-10 21:17:41 PST ---
(From update of attachment 73576)
View in context: https://bugs.webkit.org/attachment.cgi?id=73576&action=review

> WebCore/platform/graphics/mac/ComplexTextControllerCoreText.cpp:59
> +        if (runRange.location)
> +            for (unsigned i = 0; i < m_glyphCount; ++i)
> +                m_coreTextIndicesVector[i] -= runRange.location;

The WebKit style guidelines say that a multi-line block needs to be in braces. I’ll fix this when I land the patch.

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