search

NOVA-Team / NOVA-Monorepo

The core API of the NOVA voxel game modding system
https://nova-team.github.io
GNU Lesser General Public License v3.0
66 stars 23 forks source link

Allow hooking into crashreports #182

Open AEnterprise opened 9 years ago

AEnterprise commented 9 years ago

With forge you can hook intro crash reports from your TE and add additional data to the crashreport, for complex stuff this can be very helpfull to trace the cause of the issue or to replicate it

Victorious3 commented 9 years ago

Sounds too game specific for me. You have the exception, put your information there.

AEnterprise commented 9 years ago

not always, take for example a multiblock, and it throws a NPE but that's caused by some situation that could only heapen if the multiblock was build in a specific direction, if i knew how to do this (https://github.com/AEnterprise/Kinetic-Revolution/blob/master/src/main/java/kineticrevolution/multiblocks/tileEntities/TileMultiBlockBase.java#L39-L47) then it would have made tracing it down so much easier

my point is that sometimes you might want to be able to put status info in a crash report to help tracing things down as it's often hard to get the specific information from the user

RX14 commented 9 years ago

Maybe some kind of debuginfo hook that the mod provides that returns a String[] of stuff to add to the crashreport and maybe F3 menu?

AEnterprise commented 9 years ago

for the crashreport it should only be added if the block in question is the one causing the crash

RX14 commented 9 years ago

Yeah, maybe not show it on the F3 screen, but I believe that this would be possible.