Skip to content

Provide more information with BadImageReferenceException#6240

Merged
benvillalobos merged 1 commit intodotnet:masterfrom
Forgind:3246-warning-improvement
Mar 15, 2021
Merged

Provide more information with BadImageReferenceException#6240
benvillalobos merged 1 commit intodotnet:masterfrom
Forgind:3246-warning-improvement

Conversation

@Forgind
Copy link
Copy Markdown
Contributor

@Forgind Forgind commented Mar 10, 2021

Fixes #6224

Changes Made

Sent inner exception's full ToString() information instead of just its message.

Copy link
Copy Markdown
Member

@KirillOsenkov KirillOsenkov left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👍

@Forgind Forgind marked this pull request as ready for review March 12, 2021 20:34
@Forgind Forgind added the merge-when-branch-open PRs that are approved, except that there is a problem that means we are not merging stuff right now. label Mar 15, 2021
@benvillalobos benvillalobos merged commit 7a5b5e0 into dotnet:master Mar 15, 2021
@Forgind Forgind deleted the 3246-warning-improvement branch April 27, 2021 22:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

merge-when-branch-open PRs that are approved, except that there is a problem that means we are not merging stuff right now.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

RAR BadImageReferenceException needs to log the callstack and exception type

4 participants