Skip to content

More server side optimizations#195

Merged
Dream-Master merged 3 commits into
masterfrom
caedis/server-optimizations
May 27, 2026
Merged

More server side optimizations#195
Dream-Master merged 3 commits into
masterfrom
caedis/server-optimizations

Conversation

@Caedis

@Caedis Caedis commented May 26, 2026

Copy link
Copy Markdown
Member

No description provided.

@Caedis Caedis requested review from a team May 26, 2026 01:29
Comment thread src/main/scala/li/cil/oc/server/PacketHandler.scala
Comment thread src/main/scala/li/cil/oc/server/PacketHandler.scala

@DarkShadow44 DarkShadow44 left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This causes a regression: If you view a screen from 8 blocks away it's empty instead of displaying its contents. Easiest to test by writing something, going 9 blocks away and then logging out and back in.

@DarkShadow44

Copy link
Copy Markdown

@Eldrinn-Elantey Mind checking my changes regarding your comments? It's okay from my perspective now.

@Dream-Master Dream-Master enabled auto-merge (squash) May 26, 2026 20:01
@Dream-Master Dream-Master merged commit 4b04170 into master May 27, 2026
1 check passed
@Dream-Master Dream-Master deleted the caedis/server-optimizations branch May 27, 2026 02:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants