Re: [PATCH v4 0/2] MR787: notepad: Add status bar and goto line dialog

[Huw Davies (＠huw)]

Huw Davies (@huw) commented about programs/notepad/dialog.c:
> Globals.bWrapLongLines = !Globals.bWrapLongLines;
>      CheckMenuItem(GetMenu(Globals.hMainWnd), CMD_WRAP,
>          MF_BYCOMMAND | (Globals.bWrapLongLines ? MF_CHECKED : MF_UNCHECKED));
> +    SendMessageW(Globals.hMainWnd,WM_SIZE,0,MAKELPARAM(rc.right,rc.bottom));

This looks hacky.  You probably want to move the code that you're adding to the main window's WM_SIZE handler to a helper function and call that directly from here.

Also, and this applies throughout, please leave a space after each comma.

-- 
https://gitlab.winehq.org/wine/wine/-/merge_requests/787#note_7872