Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Some LVFormatter::measureText() changes #574

Open
wants to merge 3 commits into
base: master
Choose a base branch
from
Open

Some LVFormatter::measureText() changes #574

wants to merge 3 commits into from

Commits on Jun 26, 2024

  1. text: optimize tab handling 

    Find first tab as needed, and don't check beyond marker width.
    @bbshelper
    bbshelper committed Jun 26, 2024
    Configuration menu
    Copy the full SHA
    50c2178 View commit details
    Browse the repository at this point in the history

  2. text: optimize measureText()'s bidi handling 

    Calculate bidi direction on measuring, not on every char.
    @bbshelper
    bbshelper committed Jun 26, 2024
    Configuration menu
    Copy the full SHA
    07a143e View commit details
    Browse the repository at this point in the history

  3. text: simplify measureText()'s harfbuzz check 

    It doesn't seem to be necessary to set usingHarfbuzz on the first font
met: the variable doesn't change, and when it's checked, there is always
a valid font.
    @bbshelper
    bbshelper committed Jun 26, 2024
    Configuration menu
    Copy the full SHA
    e391bbb View commit details
    Browse the repository at this point in the history