[PATCH] wordpad: Make the Replace function work and theFind/Replace dialogs behave like expected.

Dmitry Timoshkov dmitry at codeweavers.com
Sun Oct 26 22:52:55 CDT 2008


"Maarten Lankhorst" <m.b.lankhorst at gmail.com> wrote:

>> +#ifndef MIN
>> +# define MIN(a,b) ((a) < (b) ? (a) : (b))
>> +#endif
>> +
>>  /* use LoadString */
>>  static const WCHAR wszAppTitle[] = {'W','i','n','e','
>> ','W','o','r','d','p','a','d',0};
>>   
> Considering you only use this macro once, wouldn't it be better to just 
> expand MIN?

There is no need to introduce MIN/MAX at all, windef.h provides min/max
(notice lower case) already.

-- 
Dmitry.



More information about the wine-devel mailing list