With the scoreboard updater thread, there can be multiple threads editing specific bits which can result in concurrency issues. This PR aims to remove those remaining concurrency issues.
Fixed a concurrency issue for sidebars without using locks, by using a second list.
There was also a potential problem with the one-list solution, which would be that technically scores (unless you put the lock around the whole newDisplayScores block ofc.)
Fixes#5578
* Escape curly braces correctly in translation strings
* Default translation fallback should not appear in the text component
* Only left braces need to be escaped
* Some adjustments
* Move the escapeBraces method to the MessageTranslator class
* Improve code readability
* Use complied static Pattern instead of Pattern.matches
* Improve some code comments
* Use regular expression complelety to escape braces instead of the inefficient, low-readablity escapeBraces method
* Add some tests about escaping curly braces in translatable strings
* Check instance availability to avoid exception during testing
* More proper workaround for direct connection using Netty 4.2
* Add check for outdated Velocity
* BungeeCord support for Netty 4.2
* Use pooled allocator
* Small cleanup: only one allocator system property check, pin comment to static commit
* Set allocator type on Standalone using netty system property
* Don't mess with allocator, both BungeeCord and Velocity set it globally
* Temporarily remove io_uring support
In order to support io_uring transport after the 4.2 update, we'd need to separately handle 4.1 and 4.2 event loop group creation
* Add compileOnly netty io_uring dependency for Geyser-BungeeCord
* Exclude old incubator io_uring dependency on geyser-standalone
---------
Co-authored-by: Camotoy <20743703+Camotoy@users.noreply.github.com>
* Better teleport handling system.
* Accept teleport right away due to how Geyser handle stuff.
* Fixed build error (oops).
* Requested changes.
* Use entity.position() instead in JavaPlayerPositionTranslator.
* Prevent elytra gliding to match Java client behavior
To-Do's:
- Firework boosting is still very broken
- see how BDS handles this mess
Other changes:
Don't send player's onGround when riding vehicles
Support null values for data components (removal of default components)
Fix possible NPE in JavaSetEquipmentTranslator with debug mode enabled
* Add bounding box height update when a player starts gliding, send sprinting state at correct time
* Properly track spin attack - should fix <https://github.com/GeyserMC/Geyser/issues/5262> - and don't allow the Bedrock client to stop gliding mid-air
* also correct pose
* temp
* Cleanup
* Send sprinting state before inputs, resolve issue mentioned in https://github.com/GeyserMC/Geyser/issues/1705
* send sprinting after inputs
* Send vehicle input before sprinting which is sent before player movement
* Slight cleanup
* Rename setDimensions -> setDimensionsFromPose
* setSpinAttack should be protected
* Resolve mobile players being unable to climb down scaffolding
* Don't send STOP_SPRINTING packet if we weren't sprinting
* Initial 1.21.80 changes
* Properly set boat rowing animation, update readme, don't apply form fix to 1.21.80
* Fix <https://github.com/GeyserMC/Geyser/issues/5512>
Forms that are closed don't need response handling
* gah generics
* compiles
* Add getOpenInventory method
* Remove debug, attempt to resolve barrel issues
* Fix some edge-cases related to lecterns
* inventory -> container renames
* Fix component check in JavaOpenBookTranslator, increase open attempts to 5
* Remove session variable used to force-close the player inventory, use confirm variable instead. Also: fix rare issues where server-side requested lectern closing didn't work for real lectern blocks
* Update core/src/main/java/org/geysermc/geyser/translator/inventory/InventoryTranslator.java
* Use closingInventory for lecterns too
* fix debug
