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

Apparently GD now returns false for empty text #93

Merged
merged 1 commit into from
Apr 25, 2021

Conversation

murrant
Copy link
Contributor

@murrant murrant commented Apr 15, 2020

We could also return [0, 0, 0, 0, 0, 0, 0, 0] but jpgraph seems to ignore errors, so this is fine

fixes: #92

We could also return [0, 0, 0, 0, 0, 0, 0, 0] but jpgraph seems to ignore errors, so this is fine
@ffflabs
Copy link
Collaborator

ffflabs commented Apr 25, 2021

(TL/DR) it's still helpful to stop the misleading error message from showing up when the font isn't the actual problem, so I'll merge

@ffflabs ffflabs merged commit 5d56a2f into HuasoFoundries:master Apr 25, 2021
@ffflabs
Copy link
Collaborator

ffflabs commented Apr 25, 2021

where are my manners 😢
@murrant thank you for the contribution!

@ffflabs
Copy link
Collaborator

ffflabs commented Apr 26, 2021

@all-contributors please add @murrant for code

@allcontributors
Copy link
Contributor

@ffflabs

I've put up a pull request to add @murrant! 🎉

@ffflabs
Copy link
Collaborator

ffflabs commented Apr 26, 2021

@all-contributors please add @murrant for code

@allcontributors
Copy link
Contributor

@ffflabs

I've put up a pull request to add @murrant! 🎉

@murrant
Copy link
Contributor Author

murrant commented Apr 27, 2021

Thanks @ffflabs :)

@murrant murrant deleted the patch-1 branch April 27, 2021 02:48
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.

cannot find fonts
2 participants