Skip to content
This repository has been archived by the owner on Aug 23, 2019. It is now read-only.

Fix #4 Improved exception reporting: Entire exception instead of message only. #6

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

BrunoJuchli
Copy link

The ExceptionReporter now prints the full exception, including stack trace, to the console.
This is intended to fix #4

@BrunoJuchli
Copy link
Author

@danielmarbach
.ToString() is actually quite neat. It will print this for an exception with inner exception:

System.InvalidOperationException: Master says you no good. Inner Exception you shall peek for ennlightement! ---> System.ArgumentException: nada nadie burranga
at Pmr.Fdms.Utilities.temp.ThrowSomeException() in c:\development\fdms\03_Source\DesktopDispatchConsole\Utilities.Tests\temp.cs:line 36
at Pmr.Fdms.Utilities.temp.RethrowThrownExceptionAsInnerException() in c:\development\fdms\03_Source\DesktopDispatchConsole\Utilities.Tests\temp.cs:line 26
--- End of inner exception stack trace ---
at Pmr.Fdms.Utilities.temp.RethrowThrownExceptionAsInnerException() in c:\development\fdms\03_Source\DesktopDispatchConsole\Utilities.Tests\temp.cs:line 32
at Pmr.Fdms.Utilities.temp.FooBar() in c:\development\fdms\03_Source\DesktopDispatchConsole\Utilities.Tests\temp.cs:line 14

I don't think there's a lot of risk that this will change - as there's lots of tools relying on this.
What did you have in mind? Can you point me to a better explicit / proper formatting code?

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants