Fabulously Optimized

Fabulously Optimized

2M Downloads

StackDeobfuscator

Madis0 opened this issue ยท 3 comments

commented

CurseForge link

https://www.curseforge.com/minecraft/mc-mods/stackdeobf

CurseForge Mod Distribution

Allowed

Modrinth link

https://modrinth.com/mod/stackdeobf

Source/other link

https://github.com/booky10/StackDeobfuscator

Mod file size

149.64 KiB + ca 7 MB downloaded afterwards (Yarn)

License

(any other license)

What it does

Maps the class names in crashes with their actual value

Why should it be in the modpack

Looks very useful for determining the cause of crashes, perhaps even moreso than MixinTrace. Could be more like #439 without the cons.

Iris dev endorsement (of the theoretical idea, not actually tested):

Seems good, since it's just converting mappings

ETF dev:

On the one hand I love it, on the other hand the standing out of non obfuscated text in stack traces is probably the single most helpful thing in identifying mod issues, if you need to identify a vanilla class then the troubleshooting is already out of the scope of a typical end user

Why shouldn't it be in the modpack

  • ETF dev's quote brings a good point, mod issues are usually visible in between the obfuscated classes
  • Untested, unsure how much it helps in practice
  • Unsure if it really helps mod devs or may cause problems too, have to be very careful

Additional details

If this proves very useful, I might even reconsider #440

commented

Useful for mod devs to debug mixin related issues in their mods as the class name can help them figure out which mixin of theirs is causing problems

Users who add other mods on top of FO can benefit from this even more, if those mods use a lot of mixins

commented

I've now tested the dev's web version of the tool out and it seems to work well, though that makes me think whether there is currently a need for this in the mod form.

The mod technically just replaces words (instead of adding new content to file like MixinTrace), and users are expected to post most crashlogs to the FO Discord anyway, so logs can be copied there if relevant.
Additionally, it may not be needed in a lot of times per ETF dev's comment, as mods' mixin names are already visible by default.