Skip to content

Draft: Packet priority removal

mug0n requested to merge mug0n/wings-xi:packet-priority-removal into master

[x] I have read, understood and agreed to the contributor's agreement. I understand that Project Wings operates under the GNU GPL licensing structure, which can be found here: https://www.gnu.org/licenses/gpl-3.0.en.html

[x] I am either the the sole copyright owner of the code or I am acting on behalf of and with the permission of the copyright holder to push the code within this merge request under the GNU GPL license. The credited party, if not myself, should be:

[x] My request includes a C++ change. I have compiled the project and run a debug build within a debugging enviornment, and fixed any issues including assertion errors. I have checked my code for, and closed any memory allocations so as not to possibly introduce a memory leak.

[-] My request includes a Lua change. I have enabled full debug logs and fixed all errors and warnings displayed on startup and during my tests, including any navmesh errors.

[-] My request includes a SQL change. I have imported all modified SQL files into my local database and verified that there are no syntax errors.

[ ] I confirm that I've run basic sanity tests on my contribution. This includes not only the above checkboxes, but also basic interactions with your code contribution in a test environment. (Such as a local install or hosted VPS.) Contributors that frequently do not test their code may not have future merge requests approved.

  • Removed the packet priority system, this will ease up cpu usage and hopefully a few packet oddities. Packet accumulation should keep packets under control.

Note: while I ran some tests locally, I unfortunately do not have a player base to test properly. Will try to invite people to run a dynamis on a test server.

Provide a "patch note" for your merge request here. This patch note will be directly copy and pasted into the #patch-notes section of the Wings Discord when this content is patched to the live server. Make sure to include whatever in-game name you would like people to see in parenthesis after the patch note.

Merge request reports