[Bug 32798] Memory leak in Path of Exile

[wine-bugs at winehq.org]

http://bugs.winehq.org/show_bug.cgi?id=32798

--- Comment #5 from Aric Stewart <aric at 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

-- 
Configure bugmail: http://bugs.winehq.org/userprefs.cgi?tab=email
Do not reply to this email, post in Bugzilla using the
above URL to reply.
------- You are receiving this mail because: -------
You are watching all bug changes.