> From: Rob Shearman [mailto:robertshear...@gmail.com] > > 2009/1/12 Alexandre Julliard <julli...@winehq.org>: > > "Ge van Geldorp" <g...@gse.nl> writes: > > > > > > Changelog: > > > NdrVaryingArrayUnmarshall() is broken on Windows > > > > If it's broken on Windows then there's no point in testing > > it, you can simply remove it. > > There's value in the function working in Wine and testing the > function stops it being inadvertently broken (probably by > me). Microsoft may release a patch to fix it or fix it in a > more recent version of Windows.
That was my reasoning too. The test itself seems valid, it's just Windows that's broken. BTW, I did some more testing, initializing each of the fields in StubMsg in turn to zero. It's StubMsg->Offset that's causing the problems. I've just submitted a patch that removes the test, Alexandre can pick whichever he prefers. Ge.