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

[Carl Eugen Hoyos]

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