http://bugs.winehq.org/show_bug.cgi?id=32798
--- Comment #5 from Aric Stewart aric@codeweavers.com 2013-01-28 08:18:52 CST --- (In reply to comment #4)
Created attachment 43361 [details] Proof of concept patch
I've added state variables to every "cache" just for the sake of symmetry, not sure if this is really necessary.
Hi,
Thanks for looking into this. I think that checking for a zero count is the correct way to do this. As for adding the _initialized flag, I can take or leave that as a concept but I do not think your patch puts it in the right place. I think we would want it to have 2 values, one for when GSUB gets intitialized and then after the cache gets expanded with GPOS. and it should get checked and set right at the top of each function.
Thanks again, -aric