FlowerChild wrote:Hey man, yeah, I was planning on PMing you that, but was totally burned out last night after wrapping this thing up for the new release. Give me a chance to have come coffee & cigarettes and I'll get that right to you :)
Take your time man, it's not to the minute ;) I doubt I'll release anything tonight, I'm deep in my own debugging. I know many of my users will be happy not to be dependent on Forge anymore though!
Well, I just PMed it to you, so no time taking required ;)
Let me know if you have any problems. Keep in mind that if you get compile errors, we may be using a different version of the MCP names or what have you, so you might have to fix that part manually.
Oh, and BTW, you'll probably figure this out just by looking at it, but all changes from the original vanilla source-files are bracketed by:
// FCMOD: foo
and
// END FCMOD
With foo being a brief explanation of what is done to the code. Generally, the original code is left in place, commented out, which makes it easier to recognize what code has changed from one version to another, and if the modifications need to be adjusted accordingly.
Anyways, hopefully we won't have to do this again for another MC release, but if we do, those comments should make the process a lot easier for you if you want to tackle it on your own next time.
May I ask you remove the "Forge sprite API" thing though? That code isn't mine (it was written by Eloraam), and I'd prefer people not think I'm trying to take credit for it given the overall political situation involved :)
I'll add Eloraam alongside you with "Original code for Forge Sprites" and put you as "Help with Forge Sprite API". Would that be fine?
Otherwise, unfortunately I've just tried adding your code to a clean MCP6 base and I'm still getting that crash (without Millénaire running). Any idea?
I'll add Eloraam alongside you with "Original code for Forge Sprites" and put you as "Help with Forge Sprite API". Would that be fine?
Sure, that sounds good man :)
Kinniken wrote:
Otherwise, unfortunately I've just tried adding your code to a clean MCP6 base and I'm still getting that crash (without Millénaire running). Any idea?
Doh! Man, that really surprises me, and unfortunately that error really doesn't mean anything to me. Any chance you can step through the code in debug and try to narrow down what's causing it?
Kinniken wrote:I'm using Eclipse, and I've used the debug tool in Eclipse in the past in other projects. I'm just not sure as to how to launch MCP from Eclipse.
You mean launch the game right? This shouldn't involve launching MCP.
If you're using the Eclipse project bundled with MCP, you should be able to launch the game in debug using F11. You'll need to set breakpoints within the code (F9) to stop execution at various points and then step through the code to find out where the crash happens.
Ah, right! I had never used the pre-configured Eclipse project, I always made my own. Ok, I'll try debuggging.
In any case, I tried the compiled classes in a Minecraft jar, and they worked fine, so I can release it :) Finding out what is going wrong in MCP is less urgent.
Kinniken wrote:Ah, right! I had never used the pre-configured Eclipse project, I always made my own. Ok, I'll try debuggging.
In any case, I tried the compiled classes in a Minecraft jar, and they worked fine, so I can release it :) Finding out what is going wrong in MCP is less urgent.
Cool stuff man. Glad you figured out a way around it :)
Hey man, quick question on Forge's texture handling - don't bother if you don't know it off the top of your head ;) Is there a simple way to check the size of the texture pack currently being used? I.e whether it's a 16x, 32x, etc?
Kinniken wrote:Hey man, quick question on Forge's texture handling - don't bother if you don't know it off the top of your head ;) Is there a simple way to check the size of the texture pack currently being used? I.e whether it's a 16x, 32x, etc?
Yikes. No idea :)
Isn't some kind of external patch required for textures beyond 16x?
Strangely enough, this time I got it to work while in dev mode using MCP without too much issues, just reusing the 1.2.3 classes you had provided with minor updates to account for method name changes. However, it crashes with a black screen when packaged:
I can't guarantee but I believe FC is not planning on updating to 1.2.4. If that's what you're talking about :P
Just a heads-up, as I'm sure he'll be glad to help.
Ok, the problem must have been with my Minecraft install, because I started again from scratch installing both ModLoader and the Forge Sprite classes manually and it's working fine.