[FFmpeg-devel] [PATCH 5/6] opus_celt: rename structures to better names and reorganize them

Rostislav Pehlivanov atomnuker at gmail.com
Wed Feb 1 15:03:23 EET 2017


On 1 February 2017 at 11:51, Michael Niedermayer <michaelni at gmx.at> wrote:

> On Wed, Feb 01, 2017 at 03:13:08AM +0000, Rostislav Pehlivanov wrote:
> > This is meant to be applied on top of my previous patch which
> > split PVQ into celt_pvq.c and made opus_celt.h
> >
> > Essentially nothing has been changed other than renaming CeltFrame
> > to CeltBlock (CeltFrame had absolutely nothing at all to do with
> > a frame) and CeltContext to CeltFrame.
> > 3 variables have been put in CeltFrame as they make more sense
> > there rather than being passed around as arguments.
> > The coefficients have been moved to the CeltBlock structure
> > (why the hell were they in CeltContext and not in CeltFrame??).
> >
> > Now the encoder would be able to use the exact context the decoder
> > uses (plus a couple of extra fields in there).
> >
> > FATE passes, no slowdowns, etc.
>
> This doesnt build on arm-linux-gnueabi-gcc-4.5
> (i think arm is unrelated here)
>
> In file included from src/libavcodec/opus_celt.c:28:0:
> src/libavcodec/opus_celt.h:134:3: error: redefinition of typedef
> ‘CeltFrame’
> src/libavcodec/opus.h:82:26: note: previous declaration of ‘CeltFrame’ was
> here
> make: *** [libavcodec/opus_celt.o] Error 1
>
>
> [...]
> --
> Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB
>
> No great genius has ever existed without some touch of madness. --
> Aristotle
>
> _______________________________________________
> ffmpeg-devel mailing list
> ffmpeg-devel at ffmpeg.org
> http://ffmpeg.org/mailman/listinfo/ffmpeg-devel
>
>
Seems older GCC versions are unhappy.
Removed the typedef from the struct definition in opus_celt.h and attached
new patch
-------------- next part --------------
A non-text attachment was scrubbed...
Name: 0001-opus_celt-rename-structures-to-better-names-and-reor.patch
Type: text/x-patch
Size: 75048 bytes
Desc: not available
URL: <http://ffmpeg.org/pipermail/ffmpeg-devel/attachments/20170201/6e133704/attachment.bin>


More information about the ffmpeg-devel mailing list