[FFmpeg-devel] [PATCH v2 00/16] Replace native DCA decoder with libdcadec based one

Michael Niedermayer michael at niedermayer.cc
Fri Jan 22 22:40:08 CET 2016


On Thu, Jan 21, 2016 at 09:44:06PM +0300, foo86 wrote:
> Updated version of the patch. I choose to split it into even smaller commits to
> make reviewing easier; you may prefer to squash it as needed.
>
> Changes since the first version:
>
> * Removed checkasm test for dcadsp
> * Removed FATE test for dca-xll (didn't check if this works)
> * Core decoder now uses butterflies_fixed() for sumdiff decoding
> * avpriv_request_sample() is now used for reporting missing features
> * Core decoder now stays in fixed point mode during intermittent XLL decoding errors
> * X96 extension is no longer parsed (and left unused) when decoding XLL
> * Minor code refactoring
>
> foo86 (16):
>   avcodec/dca: remove old decoder
>   avcodec/dca: remove unused assembly
>   avcodec/dca: remove unused data
>   tests/fate/audio: remove dca-xll test
>   tests/checkasm: remove dcadsp test
>   avcodec/dca: add REV1AUX sync word
>   avcodec/dca: add more tables
>   avcodec/dca: add math helpers and fixed point DCT
>   avcodec/synth_filter: fix whitespace
>   avcodec/synth_filter: add more filters
>   avcodec/dca: add DSP implementation
>   avcodec/dca: add generic defines
>   avcodec/dca: add core decoder
>   avcodec/dca: add EXSS parser
>   avcodec/dca: add XLL decoder
>   avcodec/dca: add new decoder based on libdcadec

intermediates dont seem to build
"tests/fate/acodec.mak:102: *** No such config: CONFIG_DCA_DECODER.  Stop."

also i would have thoght that the new decoder would be added and
the old removed once the new was in, so that all intermedate versions
work with all files

having intermedates work could help with potential future debug/
git bisect of bugs

[...]
--
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/20160122/67301a74/attachment.sig>


More information about the ffmpeg-devel mailing list