Am Montag, den 25.01.2010, 15:27 +0800 schrieb Dmitry Timoshkov:
> > The current code misdetects both Windows XP and Wine as Win9x and skips
> > some tests due to this. The SetParent tests were not passing on XP, so
> > this patch also removes the tests that fail on XP SP3.
> 
> These should be 2 separate patches.
> 
> And according to
> http://test.winehq.org/data/e2af560aa3c23a672d8680f91f3e1b8793d5827f/index_XP.html#user32:win
> SetParent tests pass under XP just fine.

No, they don't pass on Windows XP. But they also don't fail on Windows
XP, because they are not run at all, because the Win9x detection is
wrong.

If I make two separate patches out of it, we either have a no-op-patch
(changing tests that are not run), or we have an interim version that
fails tests on Windows XP. no-op-patches won't help in bisecting, as you
will always bisect to the enabling patch, and I used to think that
interim versions that fail tests are frowned upon.

Thank you for looking at the patch nevertheless,
  Michael Karcher



Reply via email to