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

More refactorings #11

Closed
wants to merge 3 commits into from
Closed

More refactorings #11

wants to merge 3 commits into from

Commits on Jul 3, 2023

  1. More refactorings 

    * Remove `freetype-rs` dependency - use (unpublished) one from sdf_glyph_renderer
* get rid of all panics (unwraps)
* add proper thiserror-based errors
* cleanup exports to be on the same top level
    @nyurik
    nyurik committed Jul 3, 2023
    Configuration menu
    Copy the full SHA
    be00b13 View commit details
    Browse the repository at this point in the history

Commits on Jul 4, 2023

  1. Make path params more generic

    @nyurik
    nyurik committed Jul 4, 2023
    Configuration menu
    Copy the full SHA
    438f584 View commit details
    Browse the repository at this point in the history

  2. Optimize combine_glyphs, add err checks 

    * combine_glyphs should not double-check hashset for values
* combine_glyphs doesn't need to have two extra iterations to compute min/max
* combine_glyphs should return None per docs on empty
    @nyurik
    nyurik committed Jul 4, 2023
    Configuration menu
    Copy the full SHA
    e563b99 View commit details
    Browse the repository at this point in the history