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

Change uses of int to size_t in MurmurHash #48562

Merged
merged 2 commits into from
Feb 7, 2023
Merged

Change uses of int to size_t in MurmurHash #48562

merged 2 commits into from
Feb 7, 2023

Conversation

gbaraldi
Copy link
Member

@gbaraldi gbaraldi commented Feb 6, 2023

Fixes #48553

@vtjnash vtjnash added backport 1.6 Change should be backported to release-1.6 backport 1.9 Change should be backported to release-1.9 labels Feb 6, 2023
@giordano giordano added the needs tests Unit tests are required for this change label Feb 6, 2023
@gbaraldi gbaraldi removed the needs tests Unit tests are required for this change label Feb 6, 2023
@oscardssmith oscardssmith merged commit 3fe69f4 into JuliaLang:master Feb 7, 2023
KristofferC pushed a commit that referenced this pull request Feb 7, 2023
@gbaraldi @KristofferC
Change uses of int to size_t in MurmurHash (#48562)
1ef6c96 
* Change uses of int to size_t in MurmurHash

(cherry picked from commit 3fe69f4)
@KristofferC KristofferC removed the backport 1.9 Change should be backported to release-1.9 label Feb 7, 2023
@KristofferC KristofferC mentioned this pull request Oct 11, 2023
Open
78 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@vtjnash vtjnash vtjnash approved these changes

Assignees

@vtjnash vtjnash

Labels
backport 1.6 Change should be backported to release-1.6
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Segfault in MurmurHash3_x64_128
5 participants
@gbaraldi @vtjnash @giordano @KristofferC @oscardssmith