[Bug 11657] Problems with MessageBox API

[wine-bugs at winehq.org]

http://bugs.winehq.org/show_bug.cgi?id=11657





--- Comment #6 from Dmitry Timoshkov <dmitry at codeweavers.com>  2008-03-08 02:10:50 ---
> +    /* Index the order the buttons need to appear to an ID* constant */
> +    static int buttonOrder[10] = { 6, 7, 1, 3, 4, 2, 5, 10, 11, 9 };

Should be 'const'

> +    for (i=1; i < 12; i++)
> +        /* No button 8 (Close) */
> +        if (i != 8) {
> +            SendDlgItemMessageW (hwnd, i, WM_SETFONT, (WPARAM)hFont, 0);
> +        }

> -    for (buttons = 0, i = 1; i < 8; i++)
> +    for (buttons = 0, i = 1; i < 12; i++)
>      {
> +        if (i == 8) continue; /* No CLOSE button */

What's so special about Close button?

> +    for (buttons = i = 0; i < (sizeof(buttonOrder) / sizeof(int)); i++) {

Should be 'sizeof(buttonOrder) / sizeof(buttonOrder[0])'


-- 
Configure bugmail: http://bugs.winehq.org/userprefs.cgi?tab=email
Do not reply to this email, post in Bugzilla using the
above URL to reply.
------- You are receiving this mail because: -------
You are watching all bug changes.