irtlab / rtptools

RTP Tools

http://www.cs.columbia.edu/irt/software/rtptools/

Other

166 stars 64 forks source link

Does not support big-endian system at the moment #71

Closed mcd500 closed 6 years ago

mcd500 commented 6 years ago

With the autotools, the endian of the system was detected by

AC_C_BIGENDIAN

and rtp.h was expecting to have WORDS_BIGENDIAN or not to properly work on bigendian system.

#ifdef WORDS_BIGENDIAN
#define RTP_BIG_ENDIAN 1
#else
#define RTP_LITTLE_ENDIAN 1
#endif

Something similar of AC_C_BIGENDIAN in configure would probably do.

janstary commented 6 years ago

Yes, I have a branch with have-bigendian.c that sets RTP_BIG_ENDIAN as used in the code. I would like to put it on top of https://github.com/columbia-irt/rtptools/pull/72 though, where the whole infrastructure gets cleaner.

janstary commented 6 years ago

fixed in https://github.com/columbia-irt/rtptools/pull/81

mcd500 commented 6 years ago

Thanks, this is one step forward for the https://github.com/columbia-irt/rtptools/issues/59