[IC2 Exp][1.2.5 to 1.12.2] Armour and Tool Status HUD

    Quote from AthenianGeneral

    Thanks for the quick update. :) I'll check out the new version and see if it complains about mystcraft.

    No problem. Your error log/config file told me exactly what went wrong and it was a very quick change to make.

    GENERATION (Pineapple + i): The first pineapple you see, copy it into your sig on any forum and add sqrt(-1) to the generation. Pineapple experiment.

    Quote from AthenianGeneral

    EDIT: Still complains about the exact same issue. :( (It's version 2.1c, same exception, deleted the config and let it regenerate, so it's definitely still an issue.) What I would do is just add a config to name all null objects something like "unnamed" or "unknown", if that's feasible. Then you could use use a try catch that supplies the configured name if it's null. Of course, I don't know how you implemented it, so your solution might differ.


    ^ Just in-case it was missed.

    Quote from AthenianGeneral

    ^ Just in-case it was missed.

    Yeah, I did miss that part (there was no new post indicator, so I didn't think to check the thread). What I did was to stick an if statement in the abbreviation save/load code where it returns the string "NULL" if the item string == null. I'll have to grab Mystcraft v0.9.1.02 and test this out.

    GENERATION (Pineapple + i): The first pineapple you see, copy it into your sig on any forum and add sqrt(-1) to the generation. Pineapple experiment.

    Quote from AthenianGeneral

    Thanks.


    (And don't forget to put mystcraft directly into the jar. It took me ages to figure out why it was refusing to run properly... :P)

    Try version 2.1d now - I've slapped in more checks for string == null and additional checks for string == "". I couldn't see any null entries for various Mystcraft objects (linking book, descriptive book, writing desk, notebook, linkbook stand) when I tried it out, so I was unable to get it to crash and test whether the fix worked, though.

    GENERATION (Pineapple + i): The first pineapple you see, copy it into your sig on any forum and add sqrt(-1) to the generation. Pineapple experiment.

    Quote from DoomFruit

    Try version 2.1d now - I've slapped in more checks for string == null and additional checks for string == "". I couldn't see any null entries for various Mystcraft objects (linking book, descriptive book, writing desk, notebook, linkbook stand) when I tried it out, so I was unable to get it to crash and test whether the fix worked, though.


    Will do, shouldn't take more than ~5 minutes.

    Single player appears to be fixed, but SMP still does it.


    Quote from AthenianGeneral

    Single player appears to be fixed, but SMP still does it.


    That error there is from Forge's horrible config file loading code, so it's beyond my control at this point. Did you start with a fresh config file for this error?

    GENERATION (Pineapple + i): The first pineapple you see, copy it into your sig on any forum and add sqrt(-1) to the generation. Pineapple experiment.

    Quote from DoomFruit

    That error there is from Forge's horrible config file loading code, so it's beyond my control at this point. Did you start with a fresh config file for this error?

    I did, and it still generated the (blank)=(blank) part that kills it. A safe way to bypass this would be some check for when you actually pull/push names into the config, making sure that null values force it to exit (do an if/else clause with the is null? It would slow it down, but probably would kill the issue in its track) and instead use a predefined value. (which preferably would be editable in the config, but that's not necessary.)

    Very important, it still generates the (blank)=(blank) in SSP, but it only crashes when you enter SMP. Whatever catch you used in SSP isn't transferring to SMP.

    Quote from AthenianGeneral

    I did, and it still generated the (blank)=(blank) part that kills it. A safe way to bypass this would be some check for when you actually pull/push names into the config, making sure that null values force it to exit (do an if/else clause with the is null? It would slow it down, but probably would kill the issue in its track) and instead use a predefined value. (which preferably would be editable in the config, but that's not necessary.)

    I could have sworn that I did that already. The if check comes at the beginning of the abbreviation subroutine, so a null value should be found and stopped immediately. This is a very strange error.

    GENERATION (Pineapple + i): The first pineapple you see, copy it into your sig on any forum and add sqrt(-1) to the generation. Pineapple experiment.

    Version 2.2 now has Gravitation Suite integration, along with an increase in the precision of the HUD-less display from 10% to 1%. Still haven't fixed the null name bug yet, though (or made the HUD position save when you press the key, lol).

    GENERATION (Pineapple + i): The first pineapple you see, copy it into your sig on any forum and add sqrt(-1) to the generation. Pineapple experiment.

    And I made a stupid copy-paste mistake on the initial v2.2 release meaning that it ignored the colour saturation value in the config and set it to maximum (thus making it a bit hard to read). If you grabbed v2.2 before this post came out, get it again (this updated 2.2 also contains an attempt by AthenianGeneral to stop the SMP Mystcraft crash, but it wasn't any more successful than my attempts).

    GENERATION (Pineapple + i): The first pineapple you see, copy it into your sig on any forum and add sqrt(-1) to the generation. Pineapple experiment.

    I put pumpkin on my head and got crash and then it kept crashing right after connecting to a server.



    Had to remove mod to be able to login again.

    Quote from matejdro

    I put pumpkin on my head and got crash and then it kept crashing right after connecting to a server.



    Had to remove mod to be able to login again.

    My guess is that you're using a localisation which contains characters beyond the standard ASCII set - this crash is when it tries to look up a character with index > 255 in the array.


    I did actually try to make it display Unicode text at the very beginning, but it kept crapping out for some reason (I don't remember exactly why, but I think it was due to the font glyphs not being cut out properly). Looks like I'll need to put Unicode back in...


    EDIT: I've tested it and it's specific to the pumpkin. Holy crap.

    GENERATION (Pineapple + i): The first pineapple you see, copy it into your sig on any forum and add sqrt(-1) to the generation. Pineapple experiment.

    Quote from matejdro

    Wow, that was fast. Thanks!

    All it took was copying the isDamageable check from the tool string builder and sticking it in the hat, shirt, trousers and shoes string builder functions - not a large change.

    GENERATION (Pineapple + i): The first pineapple you see, copy it into your sig on any forum and add sqrt(-1) to the generation. Pineapple experiment.

    And I've finally fixed the bug that makes it crash with items that don't have a name. Get v2.2d to cure your crashing.

    GENERATION (Pineapple + i): The first pineapple you see, copy it into your sig on any forum and add sqrt(-1) to the generation. Pineapple experiment.