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

removing some serious inefficiencies, fixing a change that broke compatibility with opengl < 3 #44

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

removing some serious inefficiencies, fixing a change that broke compatibility with opengl < 3 #44

wants to merge 6 commits into from

Commits on Sep 18, 2014

  1. removing completely insane and purposeless 64x horizontal scaling whi… 

    …ch is then reversed by a custom transform matrix. What were you thinking??

Reverting to using the identity matrix (although technically it isn't necessary to set this explicitly at all since it's used by default)
    @slowriot
    slowriot committed Sep 18, 2014
    Configuration menu
    Copy the full SHA
    7d0ce55 View commit details
    Browse the repository at this point in the history

  2. Avoiding unloading the Freetype library and typeface during texture_f… 

    …ont_load_glyphs() only to immediately load it again to calculate the kerning.

Explanation: texture_font_generate_kerning() is called only in one place in the entire library, and that's inside texture_font_load_glyphs().  Moving it to before the cleanup of library and face allows us to avoid a costly and completely pointless unload followed by reload.  texture_font_generate_kerning() is not declared in the headers so there is no risk of it being called elsewhere, so this change has no risk of breaking anything in any existing code.  A pointer to the existing face is passed along instead.
    @slowriot
    slowriot committed Sep 18, 2014
    Configuration menu
    Copy the full SHA
    2b99444 View commit details
    Browse the repository at this point in the history

  3. adding optional GL_ALPHA format as an alternative to the newly added … 

    …GL_RED

Older versions of OpenGL do not support GL_RED but newer ones still do support GL_ALPHA.  Rather than switching the whole thing to a deprecated format (which requires changes to the shaders also) for the sake of compatibility, I'm instead adding an #ifdef switch.
    @slowriot
    slowriot committed Sep 18, 2014
    Configuration menu
    Copy the full SHA
    0267c9b View commit details
    Browse the repository at this point in the history

  4. adding brief documentation to the defines used in texture_atlas_upload

    @slowriot
    slowriot committed Sep 18, 2014
    Configuration menu
    Copy the full SHA
    7099658 View commit details
    Browse the repository at this point in the history

  5. tweaking preprocessor defines to toggle between GL_ALPHA and GL_RED f… 

    …ormats for single channel rendering
    @slowriot
    slowriot committed Sep 18, 2014
    Configuration menu
    Copy the full SHA
    bf281ac View commit details
    Browse the repository at this point in the history

  6. adding control to enable suppression of horizontal hinting, and separ… 

    …ating its define from that of horizontal resolution to avoid confusion and increase configurability
    @slowriot
    slowriot committed Sep 18, 2014
    Configuration menu
    Copy the full SHA
    77495cc View commit details
    Browse the repository at this point in the history