Skip to content

Conversation

@neurolabusc
Copy link
Collaborator

Fixes #46.

  • Int64 values were read as 8 × int8, rather than 8 × uint8. Any value greater than 127 caused an overflow.
  • Int64 values did not correctly read sign.
  • Int64 values ignored littleEndian so failed for big-endian data.
  • The reader name has been changed from getLongAt() to getInt64At() to be explicit (in C longs are only ≥32 bits).

@rii-mango rii-mango merged commit b9bb7cd into master Feb 5, 2025
1 check passed
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

Successfully merging this pull request may close these issues.

Wrong vox_offset when reading a NIFTI-2 header

4 participants