>
> #ifdef ALLOW_MIXED
> if ( cod_info->mixed_block_flag)
> {
> if (band<2)
> block_type = 0;
> else
> block_type = cod_info->block_type;
> }
> #endif
> I search in the Lame code and the Iso code : mixed_block_flag is never
> activated, so what is the condition to have mixed blocks ?
>
Dont spend too much time working on the mixed_block code!
The ISO code (and LAME) never use it, so I doubt any of the
mixed_block code has ever been tested. From looking 128kbs mp3's produced by
FhG, I've never once seen them use mixed_blocks either, so I dont think
it is worth the effort trying to add it.
For your other suggestion,
> >
> > The sign before cos_l[13][2] has disappeared !!
> >
> The ligne to be changed is
> /* 4 */
> sum = s0 * cos_l[4][0];
> sum -= s2* cos_l[4][2];
> out[4]=sum;
>
I tried this fix and it seemed to add some low frequency background
noise into the mp3s, so I dont think it is correct? When Mike
replaced the mdct routine, I'm sure he checked that the mp3s produced
were bit-for-bit identical to the original mdct routine, and I also
verified this before adding it. Probably the sign change is
compensated somewhere else in the code.
Mark
--
MP3 ENCODER mailing list ( http://geek.rcc.se/mp3encoder/ )
