Diego Biurrun <[email protected]> writes: > On Wed, May 18, 2011 at 03:42:55PM +0100, Mans Rullgard wrote: >> These structs are only used in mpegaudiodec.c, so move them there >> and remove no longer needed #include lines from mpegaudio.h. >> >> --- a/libavcodec/mpegaudio.h >> +++ b/libavcodec/mpegaudio.h >> @@ -120,36 +95,6 @@ typedef struct MPADecodeHeader { >> - >> -/* layer 3 huffman tables */ >> -typedef struct HuffTable { >> - int xsize; >> - const uint8_t *bits; >> - const uint16_t *codes; >> -} HuffTable; > > git-grep insists that this is referenced in mpegaudiodectab.h.
So it is, and I seem to have missed that. However, that file is full of static tables, and may the lord forgive he who includes it twice. For the record, I don't believe in a lord, so the forgiving might have to wait. That said, it might make sense moving that struct to aforementioned "header" file instead. I'll send a new patch for that. -- Måns Rullgård [email protected] _______________________________________________ libav-devel mailing list [email protected] https://lists.libav.org/mailman/listinfo/libav-devel
