Skip to content

Conversation

@leMaik
Copy link
Member

@leMaik leMaik commented Dec 24, 2025

Fixes #1692 in a (hopefully) slightly more elegant way.

We could even move waterlogged and isWaterlogged() into Block where it would belong, but unfortunately the octree returns Material for blocks and changing that in 2.5.0 would break the octree plugin. Done and it doesn't break anything. 🎉

@leMaik leMaik requested a review from NotStirred December 24, 2025 23:12
@leMaik leMaik force-pushed the better-waterlogged branch 2 times, most recently from 1012da9 to e10c803 Compare December 24, 2025 23:37
Copy link
Member

@NotStirred NotStirred left a comment

Choose a reason for hiding this comment

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

nice 👍

@leMaik leMaik force-pushed the better-waterlogged branch from e10c803 to 27ac4ab Compare December 26, 2025 23:28
@leMaik leMaik force-pushed the better-waterlogged branch from 27ac4ab to 6b4ddda Compare December 26, 2025 23:56
@leMaik leMaik force-pushed the better-waterlogged branch from 6b4ddda to f95dd83 Compare December 27, 2025 00:01
@leMaik leMaik merged commit c5a37b9 into chunky-dev:master Dec 27, 2025
1 check passed
@leMaik leMaik deleted the better-waterlogged branch December 27, 2025 00:04
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.

Singletons (mainly Air#INSTANCE) and constants are incompatible with BlockSpec#maybeWaterlogged

2 participants