Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

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

Closed
mcd500 opened this issue Mar 29, 2018 · 3 comments
Closed

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

mcd500 opened this issue Mar 29, 2018 · 3 comments

Comments

@mcd500
Copy link
Contributor

mcd500 commented Mar 29, 2018

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
Copy link
Contributor

janstary commented Mar 29, 2018

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 #72 though, where the whole infrastructure gets cleaner.

@janstary
Copy link
Contributor

fixed in #81

@mcd500
Copy link
Contributor Author

mcd500 commented Mar 31, 2018

Thanks, this is one step forward for the
#59

@mcd500 mcd500 closed this as completed Mar 31, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants