On Thu, Jul 25, 2019 at 2:06 PM Steven Liu <lingjiujia...@gmail.com> wrote:
>
> Jun Zhao <mypopy...@gmail.com> 于2019年7月21日周日 下午10:32写道：
> >
> > From: vacingfang <vacingf...@tencent.com>
> >
> > Replace the same code logic with ensure_playlist(), it's will
> > help reusable blocks of code.
> >
> > Reviewed-by: Jun Zhao <barryjz...@tencent.com>
> > Signed-off-by: vacingfang <vacingf...@tencent.com>
> > ---
> > libavformat/hls.c | 10 +++-------
> > 1 files changed, 3 insertions(+), 7 deletions(-)
> >
> > diff --git a/libavformat/hls.c b/libavformat/hls.c
> > index 238ebd0..0522445 100644
> > --- a/libavformat/hls.c
> > +++ b/libavformat/hls.c
> > @@ -861,13 +861,9 @@ static int parse_playlist(HLSContext *c, const char
> > *url,
> > }
> > if (is_segment) {
> > struct segment *seg;
> > - if (!pls) {
> > - if (!new_variant(c, 0, url, NULL)) {
> > - ret = AVERROR(ENOMEM);
> > - goto fail;
> > - }
> > - pls = c->playlists[c->n_playlists - 1];
> > - }
> > + ret = ensure_playlist(c, &pls, url);
> > + if (ret < 0)
> > + goto fail;
> > seg = av_malloc(sizeof(struct segment));
> > if (!seg) {
> > ret = AVERROR(ENOMEM);
> > --
> > 1.7.1
> >
> > _______________________________________________
> > ffmpeg-devel mailing list
> > ffmpeg-devel@ffmpeg.org
> > https://ffmpeg.org/mailman/listinfo/ffmpeg-devel
> >
> > To unsubscribe, visit link above, or email
> > ffmpeg-devel-requ...@ffmpeg.org with subject "unsubscribe".
>
>
> LGTM
>
> Thanks
Will apply, Thanks
_______________________________________________
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
https://ffmpeg.org/mailman/listinfo/ffmpeg-devel
To unsubscribe, visit link above, or email
ffmpeg-devel-requ...@ffmpeg.org with subject "unsubscribe".
