[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