[PATCH 1/4] comctl32: Set updown control value to min or max in out of range case

Piotr Caban piotr at codeweavers.com
Mon Oct 28 03:58:39 CDT 2013


On 10/28/13 09:35, Alistair Leslie-Hughes wrote:
> On 17/10/2013 1:34 AM, Piotr Caban wrote:
>> ---
>>   dlls/comctl32/updown.c | 9 +++++++--
>>   1 file changed, 7 insertions(+), 2 deletions(-)
>>
>> -        } else return FALSE;
>> +        } if ((infoPtr->MaxVal > infoPtr->MinVal &&
>> infoPtr->CurVal+delta > infoPtr->MaxVal)
>              ^^^
>> +                || (infoPtr->MaxVal < infoPtr->MinVal &&
>> infoPtr->CurVal+delta < infoPtr->MaxVal)) {
>> +            delta = infoPtr->MaxVal - infoPtr->CurVal;
>
> Should the "if" be one its own line or was the else missed?
The 'else' was missed. I've sent a fix for it.

Thanks,
Piotr



More information about the wine-devel mailing list