---------- >De�: Mark Taylor <[EMAIL PROTECTED]> >� : [EMAIL PROTECTED] >Objet�: Re: [MP3 ENCODER] mdct_sub bug >Date�: Dim 3 oct 1999 20:27 > > 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. > Apologies! - cos_l[4][6] was changed in cos_l[4][2]. I hadn't seen the change of the index. Lionel -- MP3 ENCODER mailing list ( http://geek.rcc.se/mp3encoder/ )
