[MPlayer-dev-eng] [PATCH] new subopt type for malloced string

Ivo ivop at euronet.nl
Sun Jan 16 15:57:35 CET 2005


On Sunday 16 January 2005 14:35, Reimar Döffinger wrote:
> Hi,
> The attached patch moves some duplicated code to subopt_parser.c, by
> providing a new opt type, malloced, zero terminated string.
> You need to free it, and you must initialize it either to NULL or a
> malloced buffer (e.g. via strdup).
> I'd also like to extend the documentation of the test function to make
> it say: "it may also modify the argument within the limits that apply to
> initializing variables and modifying them after the parsing".
> The last part is because strarg_t.str may not be modified, and for the
> OPT_ARG_MSTRZ arg it must still point to a malloced buffer.

Looks great!

Also, I think we need a few standard validation functions, e.q. test for 
non-negative, test for [0-100] et cetera. There's already some duplicated 
code and the amount will only grow if other vo's and ao's start using this 
suboption parser.

And, if we want vf's and af's to use it, we also need something like 
OPT_ARG_INT_WITHOUT_LABEL (or something named a little better). Hmm, 
proposing a dependency for libaf, Alex B. will probably kill me for that :)

--Ivo




More information about the MPlayer-dev-eng mailing list