Elytrium / Elytrium/FastMOTD

Can't change players count via other plugin. Maybe API? + You have small bottleneck

Open
#44 2 comments 0 reactions 0 assignees View on GitHub
Dominant language
Java
Stars
72
Forks
18
PR merge metrics
No merged PRs in 30d

Description

Described the problem here:

https://github.com/PaperMC/Velocity/issues/1624

Maybe, if it can't be fixed, you'll add API to say FastMOTD take value which was set via FastMOTD API?

------------------------------------------------------------------

Also, btw, you have some small bottleneck:

Image

Don't do any math operations if value is constant. Do it only for dynamic values.
You have `+` `*` `+` `/` on each ping, it's strongly bad ☠️

Also update your benchmarks after fixing this one ^
And find similar redundant excessive calculations

Contributor guide

No contributing guide indexed for this repository

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.