[FFmpeg-soc] SVQ3 depacketizer

Martin Storsjö martin at martin.st
Thu Jul 1 21:46:18 CEST 2010


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.

Luca, Ronald, any further comments?

// Martin


More information about the FFmpeg-soc mailing list