Skip to content

packet sequential for feettrap#104

Open
0x12F wants to merge 2 commits intoNamiDevelopment:mc.1.21.11from
0x12F:mc.1.21.11
Open

packet sequential for feettrap#104
0x12F wants to merge 2 commits intoNamiDevelopment:mc.1.21.11from
0x12F:mc.1.21.11

Conversation

@0x12F
Copy link

@0x12F 0x12F commented Feb 14, 2026

No description provided.

Copy link
Member

@Kiriyaga7615 Kiriyaga7615 left a comment

Choose a reason for hiding this comment

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

basically, just not the thing i want it to be. Ill mark it as Draft to reuse sequential part later (if it works) and thats it


private final List<BlockPos> targetPositions = new ArrayList<>();
private final List<BlockPos> placedPositions = new ArrayList<>(); // todo: finish this
private final List<BlockPos> pendingSimulations = new ArrayList<>();
Copy link
Member

Choose a reason for hiding this comment

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

sorry, its just not the way it should be done

TrapFeature trapFeature = FEATURE_SERVICE.getStorage().getByClass(TrapFeature.class);
tickTimer(trapFeature);

pendingSimulations.removeIf(pos -> !MC.level.getBlockState(pos).canBeReplaced());
Copy link
Member

Choose a reason for hiding this comment

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

and this is based on client side only logic also, nahhhh

@Kiriyaga7615 Kiriyaga7615 added wontfix This will not be worked on WIP DO NOT MERGE labels Feb 15, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

DO NOT MERGE WIP wontfix This will not be worked on

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants

Comments