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

Explicitly specify int32/int64 (first installment) #10

Merged
merged 3 commits into from
Apr 20, 2023
Merged

Conversation

johnkerl
Copy link
Owner

@johnkerl johnkerl commented Apr 20, 2023

When SPFFL was written in 2004-2005, for me at the time int/unsigned were reliably 32 bits, while long long/unsigned long long were reliably 64 bits (on systems available to me at the time).

This is of course no longer true.

There's more work to be done -- this one PR of several that can be done -- which makes 32-bit/64-bit explicit in a few spots including psdes and log2.

Broader context: #1

@johnkerl johnkerl merged commit 1f88cdd into main Apr 20, 2023
@johnkerl johnkerl deleted the kerl/int32-int64 branch April 20, 2023 21:55
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.

1 participant