[MPlayer-dev-eng] [PATCH] win32 gui

[Erik Lunchpail]

> It's a PITA I know. One option could be to remove the whole
> GetCommandLine() check, but the binary would have to be named
> gmplayer.exe and started with a .bat file. This way mplayer will by
> default NOT use the gui, however there's no argv[0] check against the
> .exe exension, so it would have to be called from a .bat file from
> within the same directory by omitting the file extension (eg. START
> gmplayer) and users could shortcut to the bat file. Does this sound
> like a better solution? It would certainly remove part of the
> mplayer.c
> patching.

Actually no, that pretty much defeats the whole purpose of the IPC, not
to mention if people wanted to use it with other frontends, they'd have
to rename it back to mplayer.exe. Basically I think it has to use the
gui by default and only disabling it if calling it via slave, or by
adding a new cfg option. Using CommandLineToArvW will split up the args
and I could check each arg, but it's easier (and less code) to just
check the entire called commandline.

Regards,
Erik

__________________________________________________
Do You Yahoo!?
Tired of spam?  Yahoo! Mail has the best spam protection around 
http://mail.yahoo.com