[FFmpeg-devel] [PATCH] lavu: add escape API

Nicolas George nicolas.george at normalesup.org
Tue Mar 5 16:02:07 CET 2013

Le quartidi 14 ventôse, an CCXXI, Stefano Sabatini a écrit :
> What heuristic do you have in mind?

I do not know, maybe counting the single quotes and other special

> +enum AVEscapeMode {
> +    AV_ESCAPE_MODE_AUTO,      ///< Use auto-selected escaping mode.
> +    AV_ESCAPE_MODE_BACKSLASH, ///< Use backslash escaping.
> +    AV_ESCAPE_MODE_QUOTE    , ///< Use single-quote escaping.

Inconsistent alignment of the commas.

LGTM apart from that, thanks for humoring me.


  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/20130305/b690e1da/attachment.asc>

More information about the ffmpeg-devel mailing list