[FFmpeg-devel] [PATCH V1 1/2] lavf/hls: Cleanup the applehttp

[mypopy at gmail.com]

On Mon, Apr 15, 2019 at 11:02 PM Carl Eugen Hoyos <ceffmpeg at gmail.com> wrote:
>
> 2019-04-12 13:20 GMT+02:00, Jun Zhao <mypopydev at gmail.com>:
> > From: Jun Zhao <barryjzhao at tencent.com>
> >
> > Cleanup the applehttp as demuxer name, when use the command :
> >
> > ffmpeg -formats, get the confused information like:
> > "
> >  E hls             Apple HTTP Live Streaming
> > D  hls,applehttp   Apple HTTP Live Streaming
> > "
> > we don't use applehttp as the demuxer/muxer name usually, so
> > cleanup the applehttp and update the documents.
> >
> > After the change, get the information from "ffmpeg -formats":
> > "
> > DE hls             Apple HTTP Live Streaming
> > "
> >
> > Signed-off-by: Jun Zhao <barryjzhao at tencent.com>
> > ---
> >  doc/demuxers.texi |   20 +++++++++-----------
> >  libavformat/hls.c |    4 ++--
> >  2 files changed, 11 insertions(+), 13 deletions(-)
>
> > diff --git a/libavformat/hls.c b/libavformat/hls.c
> > index f4e4498..1b61b4b 100644
> > --- a/libavformat/hls.c
> > +++ b/libavformat/hls.c
> > @@ -2318,14 +2318,14 @@ static const AVOption hls_options[] = {
> >  };
> >
> >  static const AVClass hls_class = {
> > -    .class_name = "hls,applehttp",
> > +    .class_name = "hls demuxer",
> >      .item_name  = av_default_item_name,
> >      .option     = hls_options,
> >      .version    = LIBAVUTIL_VERSION_INT,
> >  };
> >
> >  AVInputFormat ff_hls_demuxer = {
> > -    .name           = "hls,applehttp",
> > +    .name           = "hls",
>
> This needs an entry in apichanges (which is impossible because
> there was no minor version bump) iirc, I suggest you add an
> entry to Changelog.
>
> Carl Eugen
Will add an entry to APIChange, Thanks the remind. I didn't even
realize this is a API change problem before this comment. sadly.