[MPlayer-cvslog] CVS: main/libaf af_format.c,1.26,1.27

Reimar Döffinger Reimar.Doeffinger at stud.uni-karlsruhe.de
Tue Feb 22 17:29:59 CET 2005


Hi,
On Tue, Feb 22, 2005 at 03:13:00AM +0100, Richard Felker CVS wrote:
> -      ((int8_t*)out)[i]=(int8_t)lrintf(SCHAR_MAX*((float*)in)[i]);
> +      ((int8_t*)out)[i] = (int)(127.0 * (1.0+((float*)in)[i])) - 127;

I don't quite get why it's necessary to first add 127 and then subtract
it again?? Also when this is broken, won't all other places where lrintf
is used be broken, too?
How about just improving the configure check
(-Werror-implicit-function-declaration maybe?).

Greetings,
Reimar Döffinger




More information about the MPlayer-cvslog mailing list