Skip to content

Reduce use of Unsafe.As #5554

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
Merged
merged 2 commits into from
Mar 27, 2024
Merged

Reduce use of Unsafe.As #5554

merged 2 commits into from
Mar 27, 2024

Conversation

NinoFloris
Copy link
Member

No description provided.

@NinoFloris NinoFloris changed the title Remove exact type unsafe as, it's unlikely to be faster Remove exact type unsafe as Jan 31, 2024
@NinoFloris NinoFloris marked this pull request as ready for review March 27, 2024 18:10
@NinoFloris NinoFloris closed this Mar 27, 2024
@NinoFloris NinoFloris reopened this Mar 27, 2024
@NinoFloris NinoFloris force-pushed the remove-exact-type-unsafe-as branch from cd8cc4f to 6b18540 Compare March 27, 2024 19:02
@NinoFloris NinoFloris requested a review from Brar as a code owner March 27, 2024 19:02
@NinoFloris NinoFloris changed the title Remove exact type unsafe as Reduce use of Unsafe.As Mar 27, 2024
@NinoFloris NinoFloris force-pushed the remove-exact-type-unsafe-as branch from 6b18540 to 73fbd06 Compare March 27, 2024 19:17
@NinoFloris NinoFloris merged commit 132a42d into main Mar 27, 2024
@NinoFloris NinoFloris deleted the remove-exact-type-unsafe-as branch March 27, 2024 19:25
@NinoFloris NinoFloris self-assigned this Mar 27, 2024
@NinoFloris NinoFloris added this to the 9.0.0 milestone Mar 27, 2024
NinoFloris added a commit that referenced this pull request Mar 28, 2024
@NinoFloris
Reduce use of Unsafe.As (#5554)
5344664 
(cherry picked from commit 132a42d)
@NinoFloris
Copy link
Member Author

Backported to 8.0.3 via 5344664

@roji roji modified the milestones: 9.0.0, 8.0.3 Mar 28, 2024
@NinoFloris NinoFloris mentioned this pull request Jun 24, 2024
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@vonzshik vonzshik vonzshik approved these changes

@roji roji Awaiting requested review from roji roji is a code owner

@Brar Brar Awaiting requested review from Brar Brar is a code owner

Assignees

@NinoFloris NinoFloris

Labels
cleanup
Projects
None yet
Milestone
8.0.3
Development

Successfully merging this pull request may close these issues.
3 participants
@NinoFloris @vonzshik @roji