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

Filevector doesn't work on big-endian architectures #1

Open
lckarssen opened this issue Oct 30, 2015 · 0 comments
Open

Filevector doesn't work on big-endian architectures #1

lckarssen opened this issue Oct 30, 2015 · 0 comments

Comments

@lckarssen
Copy link
Member

This is a 'liftover' of bug #5299 from the R-forge bug tracker.

The Debian build logs for big-endian machines (see URL) show that the ProbABEL checks fail on machines with that architecture. Closer inspection reveals that the checks fail on the comparison between text and binary (filevector-format) input.

Also see this discussion on the debian-mentor mailing list, especially Gert Wollny's posts: https://lists.debian.org/debian-mentors/2014/01/msg00326.html
Wollny writes:
"I dug around in the code and voila, e.g. in fvlib/frutil.cpp the
function blockWriteOrRead uses fstream.read|write to do raw data IO and
then in other parts of the code the data is just cast to the desired
type without doing any checks of endianess let alone the needed
conversions."

Since I doubt that many people will use ProbABEL/DatABEL/filevector on other (big-endian) architectures there is no hurry in fixing this. Nevertheless it's worth having this bug visible and in the back of our minds.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

1 participant