Opened 5 years ago

Closed 4 years ago

#199 closed defect (fixed)

rasdaman broken on big endian systems?

Reported by: hstamerjohanns Owned by: hstamerjohanns
Priority: minor Milestone: 8.4
Component: server Version: 8.3
Keywords: Cc:
Complexity:

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 Changed 5 years ago by hstamerjohanns

  • Owner set to hstamerjohanns
  • Status changed from new to accepted

comment:2 Changed 4 years ago by hstamerjohanns

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 Changed 4 years ago by dmisev

  • Resolution set to fixed
  • Status changed from accepted to closed

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

Note: See TracTickets for help on using tickets.