Skip to content

Commit

Permalink
Cleanup
Browse files Browse the repository at this point in the history
  • Loading branch information
Axionize committed Nov 25, 2024
1 parent 3809aed commit d0562de
Showing 1 changed file with 2 additions and 2 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -53,7 +53,7 @@ public AirLiquidPlace(GrimPlayer player) {
So that's it right? It's unfalsable?
Very close but not quite. Vanilla's client game desyncs, especially on a laggy connection where a player is breaking and placing grass 20 cps/sec in the same tick
it is possible for short grass to be interacted with even if server-side the block is air much later, and it won't be accounted for because the modification isn't recent.
This is incredibly rare, unreliable and is only triggerable if you intentionally want to false the check. Enough so that I consider a violation lvl of 2-3 to be perfect.
This is incredibly rare, unreliable and is only triggerable if you intentionally want to false the check. Enough so that I consider a violation lvl of 2-3 to be reliable and 5-6 to be autobannable
*/

@Override
Expand All @@ -80,7 +80,7 @@ public void onBlockPlace(final BlockPlace place) {
}

if (placeAgainst.isAir() || Materials.isNoPlaceLiquid(placeAgainst)) { // fail
if (flagAndAlert("tick: " + GrimAPI.INSTANCE.getTickManager().currentTick) && shouldModifyPackets() && shouldCancel()) {
if (flagAndAlert() && shouldModifyPackets() && shouldCancel()) {
place.resync();
}
}
Expand Down

0 comments on commit d0562de

Please sign in to comment.