Re: [dev] [st] [patch] If no Glyph Found, Do Not Keep a Font Loaded

2018-03-21 Thread Eric Pruitt
On Wed, Mar 21, 2018 at 05:42:07PM -0400, Gary Allen Vollink wrote: > Any request for a glyph that isn't present will load the LastResort > font, effectively killing all future fallback searches. > > I have NO idea how to fix this without opening the vacuum of space > (wow, a lot of suck). I will

Re: [dev] [st] [patch] If no Glyph Found, Do Not Keep a Font Loaded

2018-03-21 Thread Gary Allen Vollink
Notes to add to what I said above. 1) If X no longer cares if a font remains open, perhaps frc[] can be changed to a non-array. This would allow frc to continue to be used for the one-off glyph loads, without the extra maintenance of keeping track of fonts already opened. This moves the issue fr

Re: [dev] [st] [patch] If no Glyph Found, Do Not Keep a Font Loaded

2018-03-21 Thread Gary Allen Vollink
Okay, you seem to be right. Before I upgraded my Ubuntu at the end of last year I did have this issue there, too. I can assure you that this /was/ a crash on Linux, and perhaps you've just had a more recent version of the relevant X libraries since I started on this list. I know - for certain -

Re: [dev] [st] [patch] If no Glyph Found, Do Not Keep a Font Loaded

2018-03-21 Thread Hiltjo Posthuma
On Wed, Mar 21, 2018 at 02:18:53PM -0400, Gary Allen Vollink wrote: > This patch is over my previous simplification patch > (https://lists.suckless.org/dev/1803/32601.html), though it could be > reworked to go in without it. > > Problem: When working with text from many languages, it does not take

[dev] [st] [patch] If no Glyph Found, Do Not Keep a Font Loaded

2018-03-21 Thread Gary Allen Vollink
This patch is over my previous simplification patch (https://lists.suckless.org/dev/1803/32601.html), though it could be reworked to go in without it. Problem: When working with text from many languages, it does not take long to run out of font slots for Runes that do not exist within any giv

[dev] [st] [patch] Move font/rune negotiation

2018-03-21 Thread Gary Allen Vollink
The following patch is simplification only.  It puts rune lookup directly into the function that does font loading. I have a forthcoming patch that sits on top of this that fixes a rare crash situation.  This one stands alone, and because it overall removes code I feel it should be considered s