[MPlayer-dev-eng] [PATCH] Substitution of -z command line option for vo_png suboption.

Reimar Döffinger Reimar.Doeffinger at stud.uni-karlsruhe.de
Mon Jan 3 10:49:25 CET 2005


Hi,
> Here's a patch that removes the -z command line option and adds a suboption 
> 'z=<0-9>' to vo_png using the recently committed suboption parser. I also 
> updated the manual page to reflect this change.
> 
> Any comments or objections?  (Felix? Diego? :-) )

> -    {
> -	printf("PNG Unknown subdevice: %s\n",arg);
> -	return ENOSYS;
> +    if (subopt_parse(arg, subopts) != 0) {
> +        return ENOSYS;

Forgot that: while changing it anyway, I vote for using -1 instead of
ENOSYS, I really don't know why that was used - it seems that some vos
even include errno.h just to use it! After all we even have things like
VO_FALSE if somebody really wants to use some constants...

Greetings,
Reimar DÖffinger




More information about the MPlayer-dev-eng mailing list