Skip to content
This repository has been archived by the owner on Jan 4, 2023. It is now read-only.

remove unsafe keyword from from_bytes_unchecked #36

Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open

remove unsafe keyword from from_bytes_unchecked #36

wants to merge 2 commits into from

Conversation

Salka1988
Copy link
Member

@Salka1988 Salka1988 self-assigned this Dec 5, 2022
@Salka1988 Salka1988 linked an issue Dec 5, 2022 that may be closed by this pull request
Eliminate the misuse of unsafe modifier from from_bytes_unchecked #35
Open
@Salka1988 Salka1988 mentioned this pull request Dec 5, 2022
Closed
@leviathanbeak
Copy link

the unsafe comments are now redundant and should be removed

@Salka1988 Salka1988 changed the title remove ´unsafe´ keyword from ´from_bytes_unchecked´ remove unsafe keyword from from_bytes_unchecked Dec 5, 2022
@Salka1988
Copy link
Member Author

the unsafe comments are now redundant and should be removed

I will fix all that before Squash and merge

@mitchmindtree mitchmindtree mentioned this pull request Feb 10, 2023
Merged
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@leviathanbeak leviathanbeak leviathanbeak left review comments

@Voxelot Voxelot Awaiting requested review from Voxelot

@digorithm digorithm Awaiting requested review from digorithm

At least 1 approving review is required to merge this pull request.

Assignees

@Salka1988 Salka1988

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Eliminate the misuse of unsafe modifier from from_bytes_unchecked
2 participants
@Salka1988 @leviathanbeak