26 Sep
2022
26 Sep
'22
11:10 p.m.
@julliard, did you intend merge this despite the failing conformance tests (see thread above from Marvin)?
I still think the fix is "correct", i.e. this *should* workand windows is just buggy too - but the conformance tests certainly showed my fix is not really justified (since this doesn't work on windows either) and our use that led me to make this change was just getting lucky in a weird way and needs to be changed.
I hadn't just closed it because I was going to eventually salvage the tests into a fix for the memleak they revealed. But I wasn't expecting you'd to accept it after Marvin shot me down...