Skip to content
This repository was archived by the owner on Oct 14, 2026. It is now read-only.

Move boolean check in class BungeePlayerListener - #74

Closed
nickuc wants to merge 1 commit into
AuthMe:masterfrom
nickuc:patch-1
Closed

nickuc wants to merge 1 commit into
AuthMe:masterfrom
nickuc:patch-1

Conversation

@nickuc

@nickuc nickuc commented Jul 27, 2020

Copy link
Copy Markdown

No description provided.

@sgdc3

sgdc3 commented Dec 1, 2020

Copy link
Copy Markdown
Member

Any comment?

@nickuc nickuc closed this Jan 29, 2021
@ljacqu

ljacqu commented Mar 25, 2021

Copy link
Copy Markdown
Member

Moves an easy check up to avoid performing irrelevant work

@nickuc

nickuc commented Mar 25, 2021

Copy link
Copy Markdown
Author

Moves an easy check up to avoid performing irrelevant work

I do not agree. If the following pull request is implemented in the future (#48), a useless computational effort will be used, even if the feature is disabled.

@ljacqu

ljacqu commented Mar 26, 2021

Copy link
Copy Markdown
Member

I think you misread my comment ;)

@nickuc

nickuc commented Mar 26, 2021

Copy link
Copy Markdown
Author

I think you misread my comment ;)

I read it wrong. I'm sorry.

@nickuc
nickuc deleted the patch-1 branch May 19, 2021 15:22
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Development

Successfully merging this pull request may close these issues.

3 participants