[advapi32/tests #2] Don't set last error to 0 so we can check if it has changed

Paul Vriens paul.vriens.wine at gmail.com
Mon Feb 18 08:45:51 CST 2008

On Feb 18, 2008 2:17 PM, Alexandre Julliard <julliard at winehq.org> wrote:
> Paul Vriens <paul.vriens.wine at gmail.com> writes:
> > We test for a last error of 0, so we shouldn't set it to 0 before the
> > function is called. This however introduces a todo_wine.
> Is there any reason to believe that last error should be 0 on success?
> This looks more like a broken test to me.
Well no.

I agree it looks more like a broken test me as well. That's why I said
I didn't see any of the tests enter this
'if' statement.

This if statement was introduced by Rob Shearman:


I have no clue what the test does and what's wrong with it so can we
leave it like this? Or should we ask Rob to go
way back in his memory (June 2005) ;-)



More information about the wine-devel mailing list