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

Stefano Sabatini stefasab at gmail.com
Sat Aug 18 13:20:47 CEST 2012


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.
-- 
FFmpeg = Fiendish and Fierce Most Philosophical Ephemeral Glue


More information about the ffmpeg-devel mailing list