[user/tests]: Re: Fix bug that left mouse buttons swapped after tests

[Saulius Krasuckas]

* On Thu, 30 Jun 2005, Paul Vriens wrote:
> 
> > Saulius Krasuckas <saulius.krasuckas at ieee.org>
> > - Break from the loop to restore SM_SWAPBUTTON metrics.
  ...
> > --- dlls/user/tests/sysparams.c 20 Jun 2005 15:36:17 -0000      1.38
> > +++ dlls/user/tests/sysparams.c 30 Jun 2005 07:36:40 -0000
> > @@ -1011,7 +1011,7 @@ static void test_SPI_SETMOUSEBUTTONSWAP(
> >          rc=SystemParametersInfoA( SPI_SETMOUSEBUTTONSWAP, vals[i], 0,
> >                                    SPIF_UPDATEINIFILE | SPIF_SENDCHANGE );
> >          if (!test_error_msg(rc!=vals[i],"SPI_{GET,SET}MOUSEBUTTONSWAP"))
> > -            return;
> > +            break;
> 
> It's nice to break but the the end-result will be that the test fails on 
> all windows versions I could my hands on 

Test fails because of flawed logic at if-statement, which I have no time 
to debug for.  So I chose to have flawed check instead of having 
mouse-buttons swapped (which isn't so visible from the test results).

> and that the previous setting is restored. Not very useful it seams.

So if the buttons were standard mapped, then they got swapped during test 
and then test failed, you think it is OK to leave buttons swapped?