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

usize instead of u64 #553

Open
abimek opened this issue Jun 25, 2023 · 1 comment
Open

usize instead of u64 #553

abimek opened this issue Jun 25, 2023 · 1 comment

Comments

@abimek
Copy link

abimek commented Jun 25, 2023

Is there a reason why the length argument for the ProgressBar constructor takes in a u64 and not a usize. If not please give me the go ahead and I'll make a PR fixing this (if y'all want).

@djc
Copy link
Collaborator

djc commented Jun 25, 2023

Why do you want it to take a usize? It seems like the kind of thing that people might want more than 4G items for even on 32-but machines, and it also seems like a quantity that's pretty much unrelated to the word size of the target architecture.

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