Re: server: Only commit SetThreadPriority if new priority is correct (try 2)

2007-07-19 Thread Matt Jones
Ok - I was using DWORD instead of int for min & max which made them unsigned. I've corrected that now and attached an updated version of both patches (also corrected my email address in the patch) Thanks, Matt On 7/19/07, Robert Shearman <[EMAIL PROTECTED]> wrote: Matt Jones wrote: > +i

Re: server: Only commit SetThreadPriority if new priority is correct (try 2)

2007-07-19 Thread Robert Shearman
Matt Jones wrote: +if ((req->priority >= min || + req->priority == THREAD_PRIORITY_IDLE) || +(req->priority <= max || + req->priority == THREAD_PRIORITY_TIME_CRITICAL)) +thread->priority = req->priority; +} This doesn't look correct

Re: server: Only commit SetThreadPriority if new priority is correct (try 2)

2007-07-19 Thread Matt Jones
Could someone please review this? Thank you, Matt On 7/17/07, Matt Jones <[EMAIL PROTECTED]> wrote: Fixes compliance with the test I just sent in. Try 2 adds constants for realtime min/max priorities & deletes todo_wine from the fixed tests