[FFmpeg-devel] [PATCH 07/11] Remove unused ff_ac3_parse_header_full function.
Janne Grunau
janne-ffmpeg
Tue Jan 25 21:29:34 CET 2011
On Tue, Jan 25, 2011 at 10:16:54AM -0500, Justin Ruggles wrote:
> On 01/24/2011 08:29 PM, Diego Elio Petten? wrote:
>
> > ---
> > libavcodec/ac3_parser.c | 29 -----------------------------
> > libavcodec/ac3_parser.h | 11 -----------
> > 2 files changed, 0 insertions(+), 40 deletions(-)
> >
> > diff --git a/libavcodec/ac3_parser.c b/libavcodec/ac3_parser.c
> > index b844ec3..00aa372 100644
> > --- a/libavcodec/ac3_parser.c
> > +++ b/libavcodec/ac3_parser.c
> > @@ -128,35 +128,6 @@ int ff_ac3_parse_header(GetBitContext *gbc, AC3HeaderInfo *hdr)
> > return 0;
> > }
> >
> > -int ff_ac3_parse_header_full(GetBitContext *gbc, AC3HeaderInfo *hdr){
> > - int ret, i;
> > - ret = ff_ac3_parse_header(gbc, hdr);
> > - if(!ret){
> > - if(hdr->bitstream_id>10){
> > - /* Enhanced AC-3 */
> > - skip_bits(gbc, 5); // skip bitstream id
> > -
> > - /* skip dialog normalization and compression gain */
> > - for (i = 0; i < (hdr->channel_mode ? 1 : 2); i++) {
> > - skip_bits(gbc, 5); // skip dialog normalization
> > - if (get_bits1(gbc)) {
> > - skip_bits(gbc, 8); //skip Compression gain word
> > - }
> > - }
> > - /* dependent stream channel map */
> > - if (hdr->frame_type == EAC3_FRAME_TYPE_DEPENDENT && get_bits1(gbc)) {
> > - hdr->channel_map = get_bits(gbc, 16); //custom channel map
> > - return 0;
> > - }
> > - }
> > - //default channel map based on acmod and lfeon
> > - hdr->channel_map = ff_eac3_default_chmap[hdr->channel_mode];
> > - if(hdr->lfe_on)
> > - hdr->channel_map |= AC3_CHMAP_LFE;
> > - }
> > - return ret;
> > -}
> > -
> > static int ac3_sync(uint64_t state, AACAC3ParseContext *hdr_info,
> > int *need_next_header, int *new_frame_start)
> > {
> > diff --git a/libavcodec/ac3_parser.h b/libavcodec/ac3_parser.h
> > index 2f4a2ee..50b70b0 100644
> > --- a/libavcodec/ac3_parser.h
> > +++ b/libavcodec/ac3_parser.h
> > @@ -38,15 +38,4 @@
> > */
> > int ff_ac3_parse_header(GetBitContext *gbc, AC3HeaderInfo *hdr);
> >
> > -/**
> > - * Parse AC-3 frame header and sets channel_map
> > - * Parse the header up to the lfeon (channel_map in E-AC-3)
> > - * element, which is the first 52, 54 or 104 bits depending
> > - * on the audio coding mode.
> > - * @param[in] gbc BitContext containing the first 54 bits of the frame.
> > - * @param[out] hdr Pointer to struct where header info is written.
> > - * @return value returned by ff_ac3_parse_header
> > - */
> > -int ff_ac3_parse_header_full(GetBitContext *gbc, AC3HeaderInfo *hdr);
> > -
> > #endif /* AVCODEC_AC3_PARSER_H */
>
>
> E-AC-3 dependent stream parsing and decoding has still not been
> implemented and will need this, but I'm ok with removing it for now.
> Once someone shows me an E-AC-3 file with a dependent stream that's not
> just the Dolby demo I might resurrect it.
queued
Janne
More information about the ffmpeg-devel
mailing list