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 , 12 years ago
Owner: | set to |
---|---|
Status: | new → accepted |
comment:2 by , 12 years ago
comment:3 by , 12 years ago
Resolution: | → fixed |
---|---|
Status: | accepted → closed |
Thanks Heinrich, I guess we can close the ticket now.
Note:
See TracTickets
for help on using tickets.
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.