[FFmpeg-soc] SVQ3 depacketizer
Martin Storsjö
martin at martin.st
Thu Jul 1 22:08:06 CEST 2010
On Thu, 1 Jul 2010, Martin Storsjö wrote:
> On Thu, 1 Jul 2010, Josh Allmann wrote:
>
> > [lots of things fixed]
>
> Looks good to me, both of them. The long line in the doxy at the beginning
> could be wrapped, and if we'd want to be really pedantic, we perhaps
> should set extradata_size to 0 after av_freep(), since otherwise we could
> end up with extradata == NULL, but extradata_size > 0 in some situations,
> which perhaps could make other codepaths crash.
>
> But no need to resend another patch with those changes now, I'll make the
> changes when I apply them.
Actually, the url_close_dyn_buf fix needs a minor adjustment, as in the
attached patch, I'll squash that in when applying. (I see you didn't
valgrind your code this time. ;P)
// Martin
-------------- next part --------------
A non-text attachment was scrubbed...
Name: 0001-url_close_dyn_buf-Don-t-refer-to-s-max_packet_size-a.patch
Type: text/x-diff
Size: 1282 bytes
Desc:
URL: <http://lists.mplayerhq.hu/pipermail/ffmpeg-soc/attachments/20100701/e362fe59/attachment.patch>
More information about the FFmpeg-soc
mailing list