[FFmpeg-devel] [PATCH] lavf/segment: deprecate "ext" format in favor of "csv"

Stefano Sabatini stefasab at gmail.com
Fri Sep 7 10:07:38 CEST 2012


On date Saturday 2012-08-18 13:20:47 +0200, Stefano Sabatini encoded:
> On date Saturday 2012-08-18 11:46:49 +0200, Clément Bœsch encoded:
> > On Thu, Aug 16, 2012 at 06:15:10PM +0200, Stefano Sabatini wrote:
> > > The new option name is more descriptive.
> > > ---
> > >  doc/muxers.texi       |   13 ++++++++-----
> > >  libavformat/segment.c |   10 ++++++++--
> > >  2 files changed, 16 insertions(+), 7 deletions(-)
> [...]
> > > @@ -420,6 +425,7 @@ static const AVOption options[] = {
> > >      { "segment_list_size", "set the maximum number of playlist entries", OFFSET(list_size), AV_OPT_TYPE_INT,  {.dbl = 0},     0, INT_MAX, E },
> > >      { "segment_list_type", "set the segment list type",                  OFFSET(list_type), AV_OPT_TYPE_INT,  {.dbl = LIST_TYPE_UNDEFINED}, -1, LIST_TYPE_NB-1, E, "list_type" },
> > >      { "flat", "flat format",     0, AV_OPT_TYPE_CONST, {.dbl=LIST_TYPE_FLAT }, INT_MIN, INT_MAX, 0, "list_type" },
> > > +    { "csv",  "csv format",      0, AV_OPT_TYPE_CONST, {.dbl=LIST_TYPE_CSV  }, INT_MIN, INT_MAX, 0, "list_type" },
> > >      { "ext",  "extended format", 0, AV_OPT_TYPE_CONST, {.dbl=LIST_TYPE_EXT  }, INT_MIN, INT_MAX, 0, "list_type" },
> > >      { "m3u8", "M3U8 format",     0, AV_OPT_TYPE_CONST, {.dbl=LIST_TYPE_M3U8 }, INT_MIN, INT_MAX, 0, "list_type" },
> > >      { "segment_time",      "set segment duration",                       OFFSET(time_str),AV_OPT_TYPE_STRING, {.str = NULL},  0, 0,       E },
> > 
> > Why not, but I'm not sure the CSV in this context really is following the
> > rfc4180 users might expect...
> 
> I'll see if I find the motivation to implement correct CSV escaping,
> like it is done e.g. in ffprobe, assuming this is what you meant.

This one was applied.
-- 
FFmpeg = Fast and Frightening Minimal Purposeless Evanescent Geek


More information about the ffmpeg-devel mailing list