[Bug 11657] Problems with MessageBox API

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

--- Comment #6 from Dmitry Timoshkov   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])'