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.

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

check for font open errors #21

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

Conversation

geekosaur
Copy link

This may not have been done because throwIfNull can't be used directly. A wrapper using coerce works, though. Prevents xmonad from dumping core in XMonad.Prompt.printPrompt if an invalid font is specified.

This may not have been done because `throwIfNull` can't be used
directly. A wrapper using `coerce` works, though. Prevents xmonad
from dumping core in `XMonad.Prompt.printPrompt` if an invalid
font is specified.
may also reduce runtime overhead
@geekosaur
Copy link
Author

BTW, XMonad.Prompt and other consumers of this need to check for an exception now, or they'll still crash (just not with a core dump any more, and with a message in the session log). Possibly this can be done for everything in XMonad.Util.Font.

Users will need to know that they need to check for exceptions.
Copy link
Member

@slotThe slotThe left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@geekosaur
Copy link
Author

@liskin , as most recent maintainer, are we okay to bump the version and then make xmonad depend on it? Do you think we need a major version bump for anyone else who might be using it?

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.

2 participants