[FFmpeg-devel] [PATCH] qsvenc_hevc: Fix usage of ff_hevc_extract_rbsp
Michael Niedermayer
michael at niedermayer.cc
Tue Apr 26 01:52:11 CEST 2016
On Tue, Apr 26, 2016 at 12:00:16AM +0100, Derek Buitenhuis wrote:
> On 4/25/2016 11:37 PM, Michael Niedermayer wrote:
> > i cant easily test this but does this need a
> > av_freep of skipped_bytes_pos somewhere ?
> >
> > otherwise it should be ok
>
> I cannot easily test it either, unfortunately.
>
> I'm not sure why it would, though (without having looked very hard).
ive missed that the av_reallocp_array is under a skipped_bytes_pos!=NULL
check
so it should be ok
[...]
--
Michael GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB
Its not that you shouldnt use gotos but rather that you should write
readable code and code with gotos often but not always is less readable
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 181 bytes
Desc: Digital signature
URL: <http://ffmpeg.org/pipermail/ffmpeg-devel/attachments/20160426/0701af5b/attachment.sig>
More information about the ffmpeg-devel
mailing list