Opened 12 years ago

Closed 12 years ago

#199 closed defect (fixed)

rasdaman broken on big endian systems?

Reported by: Heinrich Stamerjohanns Owned by: Heinrich Stamerjohanns
Priority: minor Milestone: 8.4
Component: server Version: 8.3
Keywords: Cc:
Complexity: Medium

Description

It seems to me that even on big endian systems LITTLE_ENDIAN is always defined in endian.h which is included by stdlib.h.
Therefore all #ifdefs for LITTLE_ENDIAN in the code are probably bogus and create wrong results.
I do not have access to a big endian system, so I cannot really test this.

Is anybody running rasdaman on a big endian system?

Change History (3)

comment:1 by Heinrich Stamerjohanns, 12 years ago

Owner: set to Heinrich Stamerjohanns
Status: newaccepted

comment:2 by Heinrich Stamerjohanns, 12 years ago

Patch has been submitted. The patch does prevents preprocessor warning messages that
LITTLE_ENDIAN has been redefined.
The patch creates the same binaries on little endian systems, and therefore (is it should) does not change anything on little endian systems. However on big endian systems the code should now correctly take the #else part of the code. This has not been tested in detail, since I have no access to a big endian system.

comment:3 by Dimitar Misev, 12 years ago

Resolution: fixed
Status: acceptedclosed

Thanks Heinrich, I guess we can close the ticket now.

Note: See TracTickets for help on using tickets.