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.

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

Jump to bottom

[Rust] Replace i64 with usize for some bit utility functions #19863

Closed
asfimport opened this issue Oct 18, 2018 · 1 comment
Closed

[Rust] Replace i64 with usize for some bit utility functions #19863

asfimport opened this issue Oct 18, 2018 · 1 comment
Assignees
@sunchao
Labels
Component: Rust Type: bug
Milestone
0.12.0

Comments

@asfimport
Copy link
Collaborator

We currently use i64 in some bit utility functions. This can be replaced by usize to avoid unnecessary type conversions and invalid argument.

Reporter: Chao Sun / @sunchao
Assignee: Chao Sun / @sunchao

PRs and other links:

Note: This issue was originally created as ARROW-3549. Please see the migration documentation for further details.
@asfimport
Copy link
Collaborator Author

Krisztian Szucs / @kszucs:
Issue resolved by pull request 2784
#2784

@asfimport asfimport added this to the 0.12.0 milestone Jan 11, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@sunchao sunchao

Labels
Component: Rust Type: bug
Projects
None yet
Milestone
0.12.0
Development

No branches or pull requests
2 participants
@sunchao @asfimport