Skip to content
This repository has been archived by the owner on Apr 28, 2020. It is now read-only.

1.13.1 #50

Closed
wants to merge 5 commits into from
Closed

1.13.1 #50

wants to merge 5 commits into from

Conversation

modmuss50
Copy link

@modmuss50 modmuss50 commented Oct 2, 2018

Please review current state

Find MCP yourself I dont agree with re-hosting it, this should be changed
Compiles, loads into a world. No mods have been tested

Still to do:

  • Fix access transformers
  • Move to srg, pls
  • Fix test mod
  • Update mappings

@modmuss50
Copy link
Author

@Chocohead yeah it is lol 👍 nice spot

@@ -30,7 +30,7 @@ public ArmorMaterialImpl(String name, int maxDamageFactor, int[] damageReduction

@Override
public int getDurability(EntityEquipmentSlot slot) {
return ArmorMaterial.MAX_DAMAGE_ARRAY[slot.getIndex()] * maxDamageFactor;
return getDurability(slot) * maxDamageFactor;

Choose a reason for hiding this comment

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

Won't this just call itself recursively?

import net.minecraft.util.registry.RegistryNamespaced;
import net.minecraft.world.WorldServer;

import java.util.function.Predicate;

public abstract class Message {
public static final RegistryNamespaced<ResourceLocation, Class<? extends Message>> REGISTRY = new RegistryNamespaced<>();

Choose a reason for hiding this comment

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

Should the "message" string not be namespaced?

Copy link
Author

Choose a reason for hiding this comment

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

Should be better, I wasn't sure if it was best to add a new class or use mixin's to add the method straight to IRegistry

@ghost
Copy link

ghost commented Oct 15, 2018

How does setting up MCP for 1.13.1 work

@Chocohead
Copy link

I've got a half finished version of FG2 that is designed to take the newer FG3 config style here, theoretically all that needs finishing is GenSrgs#writeOutExcs be switched to pull from the constructors.txt rather than the old joined.exc. That would avoid any re-hosting of MCP which rubs some people the wrong way and work as a stop gap until FG3 actually works and MixinGradle plays ball with it.
I've not tested it actually trying to create a clean workspace, but it is much closer to work with the new style than stock FG2 is. Might even be able to allow Rift to have setupDecompWorkspace back if we're lucky.

@Chocohead Chocohead mentioned this pull request Nov 7, 2018
@modmuss50
Copy link
Author

@Chocohead is taking this PR over, see #58

@modmuss50 modmuss50 closed this Nov 7, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants