Skip to content

No panic at missing glyph#40

Merged
l4l merged 2 commits intomasterfrom
handle-missing-glyph
Apr 14, 2021
Merged

No panic at missing glyph#40
l4l merged 2 commits intomasterfrom
handle-missing-glyph

Conversation

@l4l
Copy link
Owner

@l4l l4l commented Apr 14, 2021

DrawTarget::draw_text calls were replaced with manual implementation: panic is replaced with logging.

Resolves #30

l4l added 2 commits April 14, 2021 21:27
DrawTarget::draw_text was manually rewritten where panic is replaced
with logging.
@l4l l4l force-pushed the handle-missing-glyph branch from 51e9909 to 7ebe8af Compare April 14, 2021 18:28
@l4l l4l merged commit bdcafcb into master Apr 14, 2021
@l4l l4l deleted the handle-missing-glyph branch April 14, 2021 18:40
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Paniced trying to unwrap Err(NoSuchGlyph)

1 participant