kernel32: Additional SetThreadPriorityTest (try 6)

James Hawkins truiken at gmail.com
Tue Jul 24 16:55:27 CDT 2007


On 7/24/07, Matt Jones <mattj at google.com> wrote:
> At this point in the test suite, SetThreadPriority has already been
> tested, so this would be duplicating the existing test. I'd be happy
> to add this in as well, but it seems unnecessary.
>

Fair enough :)  By the way, make sure to bottom-post on the wine MLs.

> On 7/24/07, James Hawkins <truiken at gmail.com> wrote:
> > On 7/24/07, Matt Jones <mattj at google.com> wrote:
> > > Reformatted tests to address James' email
> > >
> >
> > +   SetThreadPriority(curthread,min_priority);
> > +   SetLastError(0xdeadbeef);
> > +   result = SetThreadPriority(curthread,min_priority-1);
> >
> > Is there a reason you don't check the result from the first call to
> > SetThreadPriority?  If the call hypothetically fails (something you
> > have to assume can happen), then the rest of the test is not valid.
> >
> > --
> > James Hawkins
> >
>


-- 
James Hawkins



More information about the wine-devel mailing list