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

Stefano Sabatini stefasab at gmail.com
Thu Mar 7 01:49:59 CET 2013


On date Tuesday 2013-03-05 16:02:07 +0100, Nicolas George encoded:
> 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
> characters.
> 
> > +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.

Pushed, thanks for the extensive review.
-- 
FFmpeg = Friendly Formidable Mere Philosofic Entertaining Gangster


More information about the ffmpeg-devel mailing list