On Fri, Nov 29, 2019, 13:05 Sven Baars sven.wine@gmail.com wrote:
On 29-11-2019 11:57, Matteo Bruni wrote:
else
ok(!desc.Stream,·"(%u):·got·unexpected·Stream·%u.\n",·i,·desc.Stream);
Hi Matteo,
This is not true according to the tests:
Oh indeed, there is something weird going on (e.g. https://test.winehq.org/data/ec9eddf86b144cc0fcf975a9cc864cf08ba84eaf/win10_...). That suggests that somehow the "missing" field is changed by the implementation? AFAICS the local variable is supposed to be completely initialized to 0 by the test. Also of note, it looks like only the 64-bit tests are affected.
Also, if you prefer your fix, it should probably be done for the other Stream tests in that file as well.
Well, not if it's wrong... We need to figure out what's going on first. I'll give it a deeper look later today. Alexandre, please ignore the patch for the time being.
Best, Sven
Thanks for the heads up, Matteo.