-
Posts
9273 -
Joined
-
Last visited
-
Days Won
68
Everything posted by LexManos
-
IDE plugin for @SidedProxy, _factories.json, etc
LexManos replied to SoniEx2's topic in General Discussion
User was banned for, among other things, his not-so-subtle threat to hack things with a coremod if he doesn't get his way. -
@Mod as a shorthand for the mod package in _factories.json
LexManos replied to SoniEx2's topic in Suggestions
Ambiguous in the sense that this is a strictly defined "Use your class name" Where as if you start introducing variables then it just makes it more complicated. And yes FORGE already parses the mod, tho it doesn't keep any information about the package so we'd still have to look that up from the container. But this isn't about Forge it's about other things that look up this information outside of the Minecraft runtime. Seriously no, it saves you 10 seconds, but adds a ton of complexity to the system. Stop being lazy and just write out the package. If you have IDE it's literally 4 clicks. -
@Mod as a shorthand for the mod package in _factories.json
LexManos replied to SoniEx2's topic in Suggestions
No, this makes things more ambiguous. And requires parsing binary files in order to get the value that is needed. Which is bad for anyone else looking at the files. -
Logic would suggest you should register them at the same time you register Blocks, so use the Register<Block> event.
-
No, the default is fine as is. And breaks assumptions of existing code.
-
Right now there is is no better option. We could patch ItemSeeds to use the delegate system like we do for ItemBlock/ItemBlockSpecial. However as it sits any other modder would need to override the ItemSeed if they replace the Block in vanilla so there isn't any issue. You're making a problem where there isn't one.
-
Why? You want to replace the vanilla ItemSeed instances in the registry with your own thing. Who cares that you are subclassing things. You don't NEED to subclass things. Just register your override for the Item. Seriously dude, sit down, take 10 seconds. Write down what you're trying to accomplish at the end of the road. What is your mod doing that has interaction with ItemSeed. We need to understand what you want to do, not how you want to do it. http://xyproblem.info/
-
You want to override the Item, there ya go. Replace the item! What does anyone else matter?
-
Why in gods name are you overriding YOUR OWN ITEMS? Seriously dude, Explain in full detail exactly what you are trying to accomplish. Not what you're thinking you should try.
-
Do not use Object Holders for this use delegates.
-
It doesn't matter what it is called stop using random phrases and expecting others to understand them. Just describe in unambiguous terms what you're doing so that everyone can understand. As for what to do with your own code. Registry DELEGATES the exact thing we've been telling you to do for years.
-
Override the item when you override the block. Um no, they are working just fine. Quit BITCHING about things and actually HELP if you have issues. Seriously you're REALLY pissing off most of the community. Let this be your final warning, you're already banned from almost off IRC/Git Projects for your attitude. Don't make me have to ban you from here to.
-
Something To Install Mods For Minecraft Forge?
LexManos replied to A Sin Upon Reality's topic in Suggestions
Forge will never 'install mods for you' If you want to play packs, use twitch AKA Curse. Thats where most mods are hosted so its fairly easy to use. -
"I wrote a coremod you must add my coremod!" We do not accept 'suggestions' like this. Please file a suggestion correctly explaining what you want and what you are trying to achieve it for. I've removed your coremod code and your advertisement for your coremod. Don't do that.
-
There is no need for this. Also there shouldn't be any need for a mod to use root.
-
[1.12] Crafting Recipe Search Not Working
LexManos replied to SantaSenpai's topic in Support & Bug Reports
Fixed: https://github.com/MinecraftForge/MinecraftForge/commit/872d721e77a022c1ae63ba9613dbaa140c76b47b Not exactly how i'd like to do it but it works. Need to speak with cpw more about his networking code to make it better. Get the latest Forge version. -
[1.12] Crafting Recipe Search Not Working
LexManos replied to SantaSenpai's topic in Support & Bug Reports
Interesting, The system is baked no matter what you connect to so this shouldn't be an issue. Sadly I don't have any realm servers to test on. SO thats one avenue I haven't tested. -
[1.12] Crafting Recipe Search Not Working
LexManos replied to SantaSenpai's topic in Support & Bug Reports
Not a Forge issue, try without optifine, and post your log like you're supposed to -.- -
Hide vanilla item subtypes from creative tabs
LexManos replied to Jay Avery's topic in Modder Support
For your own items yes, the getSubItems is called for all tabs and you can filter the ones you want. For other's items no. -
Fire RegistryEvent.Register<IRecipe> after loading JSON recipes
LexManos replied to cad97's topic in Suggestions
It is not intended to allow you to remove recipes during the Register event. It is intended to be json driven. It just hasn't been implemented yet. But yes, the Register<IRecipe> event will eventually be fired multiple times, as it sits the current state is not determined and you should only REGISTER your shit in that event not remove others.- 1 reply
-
- 1
-
No, Biomes are a full registry, use the Register<Biome> event and just register yours as the name that you wanna override. All other methods are hacky and you shouldn't do it. But it more sounds like you just want to add a variant to the jungle biome instead of overriding it... Just make them 1/10th the weight of the vanilla one.
-
bug ItemStackHolder crashes on private field (ObjectHolder doesn't)
LexManos replied to cad97's topic in Support & Bug Reports
Yes this is kinda intentional. Honestly I think ItemStackHolder shouldn't need to exist anymore. So when I was doing the rewrite I didn't touch it. Private field support was a ADDED functionality I added when I re-worked the registry stuff. So ya, no real change here, just don't use ItemStackHolder on private fields. -
Forge 1.12 and Minecraft using different versions of JOpt
LexManos replied to fr1kin's topic in Support & Bug Reports
Right, Forgot we have to force the server to have JOpt, because Mojang believes in manually parsing the command line for the dedicated server -.- -
No, get the Java version.