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

BigEndian support #150

Open
UkooLabs opened this issue Jun 26, 2021 · 4 comments
Open

BigEndian support #150

UkooLabs opened this issue Jun 26, 2021 · 4 comments

Comments

@UkooLabs
Copy link

would it be possible to add support for BigEndian

@jkuhlmann
Copy link
Owner

For what would the be applicable? Are you having issues with endianess? Where does it break?

@UkooLabs
Copy link
Author

UkooLabs commented Jul 4, 2021

@jkuhlmann in my case Xbox 360. I have made some modifications which has made it work. But maybe a better more complete way to do it that also supports writing as only done some read side.

I'm not sure what other platforms I'll be using that will also need big endian yet.

@jkuhlmann
Copy link
Owner

Yeah, I think there aren't too many platforms left that default to big endian nowadays. The Xbox 360 seems hardly relevant when the Series X is out. 😉
Anyway, if you could share some sample code of what you fixed and how you did it, that might help me make a better decision on what we can do and if we should do something.

@UkooLabs
Copy link
Author

UkooLabs commented Sep 15, 2021

@jkuhlmann

Changes.zip

I have attached a zip showing my changes. You will see int.h which add various int types not available for XBOX Original. Then the endian helper which checks for endianness and can convert values.

Since im not 100% familular with clgtf I may of missed conversion of endianess, if you see any please let me know but loading models on a Xbox 360 seems to work fine and render.

As a note it seems the Xbox Original / 360 / X scene is opening up again and im one of a few teams bringing new dashboards / apps to those consoles as well as other platforms

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