-
-
Notifications
You must be signed in to change notification settings - Fork 492
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
feat(minecraft): migrate to 1.21.4 #4722
Conversation
# Conflicts: # src/main/kotlin/net/ccbluex/liquidbounce/features/command/builder/Parameters.kt
# Conflicts: # src/main/kotlin/net/ccbluex/liquidbounce/integration/theme/component/types/minimap/ChunkRenderer.kt
…BlueX#4881)" This reverts commit eb58a37. It added a compilation error
@@ -54,6 +61,10 @@ protected static <T extends PacketListener> void handlePacket(Packet<T> packet, | |||
private void hookSendingPacket(Packet<?> packet, final CallbackInfo callbackInfo) { | |||
final PacketEvent event = new PacketEvent(TransferOrigin.SEND, packet, true); | |||
|
|||
if (packet instanceof PlayerMoveC2SPacket) { | |||
this.sendInternal(new ClientTickEndC2SPacket(), null, true); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
we're flagging timer with this one 🔥
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I also think scripts should have the control to not send the packet.
@@ -22,3 +14,11 @@ fun Fog.copy( | |||
shape, color.r, | |||
color.g, color.b, color.a) | |||
} | |||
|
|||
// could be merged into `Color4b` as generics, but not sure if izuna would be fine with that. | |||
data class FloatColor(val r: Float, val g: Float, val b: Float, val a: Float) { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This can replaced by /255f when constructing the Fog object.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
but we would lose some precision, not sure if anyone cares 💀
// color.b.toFloat(), color.a.toFloat() | ||
// ) | ||
} | ||
|
||
fun Fog.copy( |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I don't like the idea of this.
I'd prefer it when you set the states directly (if possible) in the render system.
Or directly construct the Fog object without creating a new Fog object for each value. Instead you could just construct one and default everything that has not changed to the original.
This reverts commit e77a68d.
05e7a19
to
c482931
Compare
wtf |
OH SHIT |
…` when true sight is required basically a typo, the only reason this wasn't fixed in d84c68f is that I still thought it was "isInvisibleTo" and not "isVisibleTo".
Conflicts |
konflicts |
What rendering problems? |
As I said in the message, ESP, Tracers, etc (whatever LiquidBounce renders in the world), are offset from where they're supposed to be. |
This PR is probably going to get superseded by @ccetl's Minecraft 1.21.4 PR whenever it comes out. |
Superseded by #4980 because of the rendering issues. |
PR
A list from the top of my head:
fallFlying
toglide
(:skull:)Useful tools / utilities:
@Inject
,@Redirect
, access widener stuff (the Minecraft Development plugin lets you copy AW code for functions when something isn't accessible though), etc.)Some broken things
The actual diff in question (ignores merging upstream crap): nextgen...DataM0del:LiquidBounce:feat/minecraft/1.21.3