Re: bug in xfont xlfd rounding code

2009-01-29 Thread James Cloos
I took a look at the various versions of ieee754.h in glibc. The most significant difference is in long doubles. (As an example, IBM uses a pair of doubles, one for the most significant bits and one for the least significant bits, but each is usable as is as a double.) In total, there are 5

bug in xfont xlfd rounding code

2009-01-28 Thread Dave Airlie
Hi guys, Maybe someone understand wtf the code in libXfont/util/fontxlfd.c:xlfd_round_double is all about, but the results were different on different endian machines due to the code being hardcoded for little endian. I reimplemented it for other endian, however I'm unsure about the endian

Re: bug in xfont xlfd rounding code

2009-01-28 Thread walter harms
Dave Airlie schrieb: Hi guys, Maybe someone understand wtf the code in libXfont/util/fontxlfd.c:xlfd_round_double is all about, but the results were different on different endian machines due to the code being hardcoded for little endian. I reimplemented it for other endian, however

Re: bug in xfont xlfd rounding code

2009-01-28 Thread Eirik Byrkjeflot Anonsen
Dave Airlie airl...@gmail.com writes: Hi guys, Maybe someone understand wtf the code in libXfont/util/fontxlfd.c:xlfd_round_double is all about, but the results were different on different endian machines due to the code being hardcoded for little endian. Which is to say x86 layout. I

Re: bug in xfont xlfd rounding code

2009-01-28 Thread Michel Dänzer
On Wed, 2009-01-28 at 21:48 +1000, Dave Airlie wrote: Maybe someone understand wtf the code in libXfont/util/fontxlfd.c:xlfd_round_double is all about, but the results were different on different endian machines due to the code being hardcoded for little endian. I reimplemented it for

Re: bug in xfont xlfd rounding code

2009-01-28 Thread Dave Airlie
On Thu, Jan 29, 2009 at 2:17 AM, Michel Dänzer mic...@daenzer.net wrote: On Wed, 2009-01-28 at 21:48 +1000, Dave Airlie wrote: Maybe someone understand wtf the code in libXfont/util/fontxlfd.c:xlfd_round_double is all about, but the results were different on different endian machines due to