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

[Maarten Lankhorst]

Hi Elias,

Please don't send mail to wine-devel and wine-patches at the same time, 
if you want feedback, send it to wine-devel, if you want it committed, 
send it to wine-patches

Elias Benali schreef:
> diff --git a/programs/wordpad/wordpad.c b/programs/wordpad/wordpad.c
> index d06b2d9..434b6fd 100644
> --- a/programs/wordpad/wordpad.c
> +++ b/programs/wordpad/wordpad.c
> @@ -49,6 +49,10 @@
>  # define U3(x) (x)
>  #endif
>
> +#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?

Cheers,
Maarten.