[FFmpeg-devel] [PATCH 16/19] avformat/hls: factor identical playlist allocations out of parse_playlist

Michael Niedermayer michaelni at gmx.at
Sat Jan 4 00:23:14 CET 2014


On Fri, Jan 03, 2014 at 04:21:40PM +0200, Anssi Hannula wrote:
> Signed-off-by: Anssi Hannula <anssi.hannula at iki.fi>
> ---
>  libavformat/hls.c | 36 ++++++++++++++++++++++--------------
>  1 file changed, 22 insertions(+), 14 deletions(-)

maybe you want to use ret < 0 checks for the errors (as thats whats
used more commonly)

either way LGTM

[...]
-- 
Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

Frequently ignored answer#1 FFmpeg bugs should be sent to our bugtracker. User
questions about the command line tools should be sent to the ffmpeg-user ML.
And questions about how to use libav* should be sent to the libav-user ML.
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 198 bytes
Desc: Digital signature
URL: <http://ffmpeg.org/pipermail/ffmpeg-devel/attachments/20140104/447c3c9b/attachment.asc>


More information about the ffmpeg-devel mailing list