Skip to content

[crc64] Change the hash prefix back to crc64 #628

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

Merged
merged 1 commit into from
Apr 16, 2020

Conversation

grendello
Copy link
Contributor

Context: 9b88ce7
Context: 7fe3a11

7fe3a11 fixed the new CRC64 calculation algorithm so that it properly
processes all the data, which had the (expected) effect of generating
the same checksums for the same data as those generated before 9b88ce7

However, 7fe3a11 missed changing the new c64r2 hash prefix back to
crc64, thus this commit to make the change.

The change will remove any backward incompatibility with code that may
rely on the old hash format, including Xamarin.Android tests.

Context: 9b88ce7
Context: 7fe3a11

7fe3a11 fixed the new CRC64 calculation algorithm so that it properly
processes all the data, which had the (expected) effect of generating
the same checksums for the same data as those generated before 9b88ce7

However, 7fe3a11 missed changing the new `c64r2` hash prefix back to
`crc64`, thus this commit to make the change.

The change will remove any backward incompatibility with code that may
rely on the old hash format, including Xamarin.Android tests.
@jonpryor jonpryor merged commit 6d7266d into dotnet:master Apr 16, 2020
@grendello grendello deleted the faster-crc64 branch April 16, 2020 15:39
jonpryor pushed a commit that referenced this pull request Apr 22, 2020
Context: 9b88ce7
Context: 7fe3a11

7fe3a11 fixed the new CRC64 calculation algorithm so that it properly
processes all the data, which had the (expected) effect of generating
the same checksums for the same data as those generated before
commit 9b88ce7.

However, 7fe3a11 missed changing the new `c64r2` hash prefix back to
`crc64`, thus this commit to make the change.

The change will remove any backward incompatibility with code that may
rely on the old hash format, including Xamarin.Android tests.
@jpobst jpobst added this to the 10.4 (16.7 / 8.7) milestone Apr 23, 2020
@github-actions github-actions bot locked and limited conversation to collaborators Apr 13, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants