[FFmpeg-devel] [PATCH] fix wc3 xan decoder
Michael Niedermayer
michaelni
Sun Mar 2 00:11:42 CET 2008
On Sat, Mar 01, 2008 at 02:19:28PM -0800, Mike Melanson wrote:
> Michael Niedermayer wrote:
> > not in the patch, the patch works around it by setting data[0] to zero
> > dont copy it and it will be zero (from release_buffer)
> >
> > That is i think your patch is a hack not a fix :)
>
> I have been looking at the problem some more. For this codec, I need to
> keep the previous frame data for MC. Right now, I have data structures
> for current and previous frames allocated as part of the codec's private
> structure. Should I turn those into structure pointers that are
> allocated and freed during _init() and _end() functions?
Thats an option.
A AVFrame pics[2]; could also be used to avoid the malloc/free.
but the simplest solution would be
FFSWAP(AVFrame, s->currentpic, s->lastpic);
[...]
--
Michael GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB
Observe your enemies, for they first find out your faults. -- Antisthenes
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 189 bytes
Desc: Digital signature
URL: <http://lists.mplayerhq.hu/pipermail/ffmpeg-devel/attachments/20080302/45ef3418/attachment.pgp>
More information about the ffmpeg-devel
mailing list