[MPlayer-dev-eng] [PATCH rev] windows gui about box

Ingo Brückl ib at wupperonline.de
Mon Aug 29 02:05:28 CEST 2011


Stephen Sheldon wrote on Sun, 28 Aug 2011 23:32:07 +0000 (UTC):

> +static void display_about_box(HWND hWnd)
> +{
> +    char about_msg[512];
> +    snprintf(about_msg, sizeof(about_msg), MP_TITLE, "MPlayer");
> +    MessageBox(hWnd, about_msg, "About", MB_OK);
> +}

I'd prefer putting the statements right into handlemsg(), avoiding an extra
function.

Besides that, it's OK and I will add MP_TITLE to the X11/GTK GUI as well.

Ingo


More information about the MPlayer-dev-eng mailing list