Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

StackDeobfuscator #605

Open
Madis0 opened this issue Mar 22, 2023 · 1 comment
Open

StackDeobfuscator #605

Madis0 opened this issue Mar 22, 2023 · 1 comment
Labels
feedback/testers wanted Issues that could use more feedback from the users mod New Fabric mod to include on hold Mods that are waiting for something before inclusion

Comments

@Madis0
Copy link
Member

Madis0 commented Mar 22, 2023

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

@Madis0 Madis0 added mod New Fabric mod to include WIP Mods that are experimental and will not be included until considered "stable enough" labels Mar 22, 2023
@Madis0 Madis0 added this to To do in 4.7.0 Mar 25, 2023
@Madis0 Madis0 added feedback/testers wanted Issues that could use more feedback from the users and removed WIP Mods that are experimental and will not be included until considered "stable enough" labels Mar 25, 2023
@Madis0 Madis0 removed this from To do in 4.7.0 Mar 29, 2023
@Madis0 Madis0 added on hold Mods that are waiting for something before inclusion feedback/testers wanted Issues that could use more feedback from the users and removed feedback/testers wanted Issues that could use more feedback from the users on hold Mods that are waiting for something before inclusion labels Apr 21, 2023
@Madis0 Madis0 added the on hold Mods that are waiting for something before inclusion label May 7, 2023
@TechPro424
Copy link

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

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feedback/testers wanted Issues that could use more feedback from the users mod New Fabric mod to include on hold Mods that are waiting for something before inclusion
Projects
None yet
Development

No branches or pull requests

2 participants