[Ffmpeg-devel] [PATCH] Chinese AVS video decoder
Stefan Gehrer
stefan.gehrer
Sun Jul 2 18:05:01 CEST 2006
Von: Michael Niedermayer
>
> i didnt suggest to change the prediction functions, my suggestion was just
> to not copy the contents of left/top_border* into yet another buffer but
> instead pass a pointer to them ...
>
Oh, I see. I will look into that.
>
> thats ok, but making all functions in cavsdsp non static and initalizing
> DSPContext with them in dsputil.c is messy, they should stay static and
> the init should be done in a function in cavsdsp.c
>
> furthermore your patch mixes cosmetical changes (variable renamings,
> comment
> changes) and functional changes, such patches are not acceptable
> ive commited the changes to mpeg12* and avcodec.h as they are independant
> of the rest and they didnt mix cosmetic & functional changes
>
How is this for the functional part now? Eliminates cavsdsp.h and
leaves only five functions non-static in dsputil.c, all prefixed with ff_.
Regards
Stefan Gehrer
--
"Feel free" ? 10 GB Mailbox, 100 FreeSMS/Monat ...
Jetzt GMX TopMail testen: http://www.gmx.net/de/go/topmail
-------------- next part --------------
A non-text attachment was scrubbed...
Name: cavsdiff4
Type: application/octet-stream
Size: 22410 bytes
Desc: not available
URL: <http://lists.mplayerhq.hu/pipermail/ffmpeg-devel/attachments/20060702/f70a6186/attachment.obj>
More information about the ffmpeg-devel
mailing list