Skip to content

correction for MMG5_swapbin_int() #322

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.

Sign up for GitHub

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

Open
mpotse wants to merge 2 commits into
base: develop
Choose a base branch
from
Open

correction for MMG5_swapbin_int() #322

Changes from all commits
Commits
Show all changes
2 commits
Select commit Hold shift + click to select a range
5cbc30a
corrected MMG5_swapbin_int() for the case MMG5_int has a different si…
mpotse Jan 31, 2026
cb028f0
comment on this (currently unused) function
mpotse Jan 31, 2026
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
13 changes: 10 additions & 3 deletions src/common/inout.c
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -57,17 +57,24 @@ int MMG5_swapbin(int sbin)
/**
* swap bytes if needed (conversion from big/little endian toward little/big
* endian)
*
* NOTE this function is probably not useful for i/o because integer sizes
* in files are fixed and the size of MMG5_int is configurable; it is meant
* only for internal usage. This is perhaps some litter remaining from the
* interoduction of a configurable integer size. If it is ever needed, it
* should probably be MMG5_swapbin_int64(int64_t sbin).
*
*/
MMG5_int MMG5_swapbin_int(MMG5_int sbin)
{
MMG5_int out;
char *p_in = (char *) &sbin;
char *p_out = (char *) &out;
int8_t i;
int i, Nbytes=sizeof(MMG5_int);

for(i=0;i<8;i++)
for(i=0;i<Nbytes;i++)
{
p_out[i] = p_in[7-i];
p_out[i] = p_in[Nbytes-1-i];
}

return out;
Expand Down