search

fancho29 / garglk

Automatically exported from code.google.com/p/garglk
Other
0 stars 0 forks source link

INI file font names with spaces #103

Closed GoogleCodeExporter closed 8 years ago

GoogleCodeExporter commented 8 years ago 
Using font names containing spaces (like "DejaVu Sans Mono") doesn't work in
the INI file. The attached patch changes parsing for the font directives so
that it works.

Drawback of the patch: White space preceding the font name is no longer
ignored. This is probably easy to improve :)

Original issue reported on code.google.com by leftda...@gmail.com on 30 Mar 2010 at 5:24

Attachments:

GoogleCodeExporter commented 8 years ago 
Fontconfig will load fonts with spaces in the name, if they are installed to a 
system
location that it has cached.

Still, it's useful to handle this case in the event that we have to fall back to
loading fonts with Freetype.

Let me test your patch - the config file parsing always confuses me but I think 
I see
what's going on.

Original comment by bcressey@gmail.com on 31 Mar 2010 at 12:27

GoogleCodeExporter commented 8 years ago 
This was fixed in r363.

Original comment by bcressey@gmail.com on 5 Aug 2010 at 10:14