Skip to content

Fix read_uint32 and read_uint64 for big-endian machines, potentially improving performance#19

Merged
tidwall merged 1 commit intotidwall:mainfrom
Kontinuation:fix-parsing-wkb
Aug 13, 2025
Merged

Fix read_uint32 and read_uint64 for big-endian machines, potentially improving performance#19
tidwall merged 1 commit intotidwall:mainfrom
Kontinuation:fix-parsing-wkb

Conversation

@Kontinuation
Copy link
Copy Markdown
Contributor

@Kontinuation Kontinuation commented Aug 11, 2025

Please do not open a pull request without first filing an issue and/or discussing the feature directly with me.

Please ensure you adhere to every item in this list

  • This PR was pre-approved by the project maintainer
  • I have self-reviewed the code
  • I have added all necessary tests (already covered by existing tests)

Describe your changes

Fixes byte-swapping for parsing WKBs on big-endian machines, and potentially improve performance of parsing big-endian WKBs on little-endian machines.

Issue number and link

#18

@tidwall tidwall merged commit 2c5102c into tidwall:main Aug 13, 2025
1 check passed
@tidwall
Copy link
Copy Markdown
Owner

tidwall commented Aug 13, 2025

Very nice. Thank you again 😊

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.

2 participants