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

Fix random target selector - Fixes #574 #575

Merged

aramperes merged 1 commit into GlowstoneMC:master from kaenganxt:randomTargetSelector

Oct 19, 2017

Member

kaenganxt commented Oct 18, 2017

This PR fixes the IndexOutOfBoundsException when using the @r selector on a single player as stated in #574.

The commit also reuses the random object for performance reasons.


          Fix random selector

2f0ea8d

aramperes approved these changes

View reviewed changes

aramperes merged commit 62f43a2 into GlowstoneMC:master

Member

aramperes commented Oct 19, 2017

Thank you for your contribution!

kaenganxt deleted the randomTargetSelector branch

December 10, 2017 14:04

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet