[FFmpeg-devel] [PATCH] opt: implement av_opt_set_from_string().

Nicolas George nicolas.george at normalesup.org
Fri Aug 24 01:14:32 CEST 2012


Le quartidi 4 fructidor, an CCXX, Stefano Sabatini a écrit :
> I'm not sure if we should drop the possibility to specify
> separators. Indeed while we're not using the feature, it could be
> useful to the application layer (and we may need to specify different
> option separators in different contexts, e.g. "," for protocol
> options).

The thing is that preserving the quoting rules while allowing to set the
delimiters makes the code very clumsy.

It would make the code much simpler if we decide that the key can not be
escaped, and has a limited length for that matter. Since the key is actually
the name of an option, it never needs to be escaped, to contain special
characters or to be overly long, so this is in no way crippling the
features.

Regards,

-- 
  Nicolas George
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 198 bytes
Desc: Digital signature
URL: <http://ffmpeg.org/pipermail/ffmpeg-devel/attachments/20120824/d17dbc41/attachment.asc>


More information about the ffmpeg-devel mailing list