-
-
Notifications
You must be signed in to change notification settings - Fork 27
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
added nocom crash :D #3
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Found some minor things
import java.util.Random; | ||
|
||
public class NoComCrash extends Module{ | ||
final Random r = new Random(); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Tf
|
||
public class NoComCrash extends Module{ | ||
final Random r = new Random(); | ||
int i = 0; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
What's this for
|
||
private final Setting<Boolean> autoDisable = sgGeneral.add(new BoolSetting.Builder() | ||
.name("auto-disable") | ||
.description("Disables module on kick.") |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I'd rather write "Disable the module when disconnected" or smth
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
i would remove this setting entirely because it is not being used at all
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
:O didn't even notice that LOL
looks mostly fine. could you remove the unused variables and fix the formatting slightly (remove the 14 spaces and add a space before |
i have fixed my idiocy |
No description provided.