[MPlayer-dev-eng] [PATCH] Option '-display' for GUI
Reimar.Doeffinger at gmx.de
Thu Jun 2 17:01:36 CEST 2011
On Thu, Jun 02, 2011 at 02:31:27PM +0200, Ingo Brückl wrote:
> Reimar Döffinger wrote on Wed, 1 Jun 2011 21:40:58 +0200:
> >> Pass argv program name to guiInit() and thus to gtkInit().
> >> This will allow gtk_init() to pass arguments for initialization.
> > Doesn't make sense, why would GTK need the program name?
> > Just hardcoding "gmplayer" should work just fine.
> Yes, hardcoding works, but normally gtk_init() gets main()'s argc/argv, so I
> felt more comfortable to pass the original value.
> GTK's parsing routine (actually the called GLib parsing routine) uses argv
> to set a program name with g_set_prgname() for GLib. However, it only uses
> the basename of argv.
Well, even that is not used for anything useful, is it?
> > Not that I mind passing the correct value but the commit
> > message should make sense.
> I'd prefer to pass the correct value, but in what way does the commit message
> make no sense?
It says "this allows" however as I pointed out it isn't necessary for
that. A commit message that claims something that is just not true is
simply very bad.
More information about the MPlayer-dev-eng