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

Make Paper Safe Again #4913

Merged
merged 1 commit into from Dec 19, 2020
Merged

Make Paper Safe Again #4913

merged 1 commit into from Dec 19, 2020

Conversation

MiniDigger
Copy link
Member

@MiniDigger MiniDigger commented Dec 19, 2020 •
edited

Removes unused unsafe utils, as suggested by billy

(they cause issues if you compile with java 11, and dum warnings)

@MiniDigger MiniDigger requested a review from a team as a code owner December 19, 2020 11:12
Proximyst
Proximyst approved these changes Dec 19, 2020
View reviewed changes
Copy link
Contributor

@Proximyst Proximyst left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM! 🎉

@kashike kashike added type: security Issue relates to the security of the server. status: accepted Disputed bug is accepted as valid or Feature accepted as desired to be added. we must go deeper labels Dec 19, 2020
@Proximyst Proximyst merged commit de31e8c into PaperMC:master Dec 19, 2020
@MiniDigger MiniDigger deleted the make-paper-great-again branch December 19, 2020 20:49
ExcessiveAmountsOfZombies pushed a commit to ExcessiveAmountsOfZombies/Paper that referenced this pull request Apr 12, 2021
@MiniDigger @ExcessiveAmountsOfZombies
Remove Paper's UnsafeUtils (PaperMC#4913)
3c831ea
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@kashike kashike kashike approved these changes

@Proximyst Proximyst Proximyst approved these changes

Assignees
No one assigned
Labels
status: accepted Disputed bug is accepted as valid or Feature accepted as desired to be added. type: security Issue relates to the security of the server.
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
3 participants
@MiniDigger @kashike @Proximyst