Skip to content
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

1.20 port #813

Open
wants to merge 15 commits into
base: 1.20
Choose a base branch
from
Open

1.20 port #813

wants to merge 15 commits into from

Conversation

TheidenHD
Copy link

No description provided.

@Wolbobus
Copy link

If I'm not mistaken, this port works for 1.20. Would you be against uploading it yourself to curseforge until Vazkii is active on it?


cadAssembler = new BlockCADAssembler(Block.Properties.copy(metalBehaviour).strength(5, 10).sound(SoundType.METAL).noOcclusion());
programmer = new BlockProgrammer(Block.Properties.copy(metalBehaviour).strength(5, 10).sound(SoundType.METAL).noOcclusion());
cadAssembler = new BlockCADAssembler(Block.Properties.of().mapColor(MapColor.METAL).instrument(NoteBlockInstrument.IRON_XYLOPHONE).requiresCorrectToolForDrops().strength(5, 10).sound(SoundType.METAL).noOcclusion());
Copy link
Member

Choose a reason for hiding this comment

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

Note: Blocks don't seem to have proper tools set up for breaking, conjured blocks are no longer instant punch

Copy link
Author

Choose a reason for hiding this comment

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

@Kamefrede addet it in 85ef344
I don't know what his goal was but I think there should be at least a Tool that breaks them faster. Fixing it is not a problem but I neat to know what the goal is.

Copy link
Contributor

Choose a reason for hiding this comment

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

It was just to get something quick and dirty out, I intended on fixing it later

Copy link
Author

Choose a reason for hiding this comment

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

I removed the tool requirement.

@@ -116,7 +113,7 @@ public static void register(RegisterEvent evt) {

evt.register(ForgeRegistries.Keys.MENU_TYPES, helper -> {
helper.register(ForgeRegistries.BLOCKS.getKey(cadAssembler), containerCADAssembler);
DistExecutor.safeRunWhenOn(Dist.CLIENT, () -> () -> {
DistExecutor.unsafeRunWhenOn(Dist.CLIENT, () -> () -> {
Copy link
Member

Choose a reason for hiding this comment

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

Is unsafe necessary?

Copy link
Author

@TheidenHD TheidenHD Jul 31, 2024

Choose a reason for hiding this comment

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

Yes
Caused by: java.lang.RuntimeException: Unsafe Referent usage found in safe referent method
The olde runWhenOn function just wraps unsafeRunWhenOn.

@Deprecated
public static void runWhenOn(Dist dist, Supplier toRun) {
    unsafeRunWhenOn(dist, toRun);
}

https://forums.minecraftforge.net/topic/96761-1165-properly-using-distexecutor-with-arguments/

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