On Thu, Jul 25, 2019 at 2:06 PM Steven Liu <[email protected]> wrote: > > Jun Zhao <[email protected]> 于2019年7月21日周日 下午10:32写道： > > > > From: vacingfang <[email protected]> > > > > Replace the same code logic with ensure_playlist(), it's will > > help reusable blocks of code. > > > > Reviewed-by: Jun Zhao <[email protected]> > > Signed-off-by: vacingfang <[email protected]> > > --- > > 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 > > [email protected] > > https://ffmpeg.org/mailman/listinfo/ffmpeg-devel > > > > To unsubscribe, visit link above, or email > > [email protected] with subject "unsubscribe". > > > LGTM > > Thanks Will apply, Thanks _______________________________________________ ffmpeg-devel mailing list [email protected] https://ffmpeg.org/mailman/listinfo/ffmpeg-devel
To unsubscribe, visit link above, or email [email protected] with subject "unsubscribe".
