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

Make ExceptionLayoutRenderer more extensible #430

Merged
2 commits merged into from
Nov 16, 2014
Merged

Make ExceptionLayoutRenderer more extensible #430

2 commits merged into from
Nov 16, 2014

Conversation

SurajGupta
Copy link
Contributor

Right now there's no easy way to tweak how a particular layout token (i.e. Message or Data) is rendered by extending ExceptionLayoutRenderer, because the Append... methods are private static

Right now there's no easy way to tweak how a particular layout token (i.e. Message or Data) is rendered by extending ExceptionLayoutRenderer, because the Append... methods are private static
- Convert Append... methods from private static to protected virtual
- Change ParseMethodNameFromStackTrace from private to protected
@ghost
Copy link

ghost commented Oct 15, 2014

Thanks for the pull request, it looks good. I'll merge it soon

@SurajGupta
Copy link
Contributor Author

Thank you!

ghost pushed a commit that referenced this pull request Nov 16, 2014
Make ExceptionLayoutRenderer more extensible
@ghost ghost merged commit 1c4d3d7 into NLog:master Nov 16, 2014
@ghost ghost added the enhancement Improvement on existing feature label Nov 16, 2014
@ghost ghost added this to the 3.1.1 milestone Nov 16, 2014
This pull request was closed.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement Improvement on existing feature
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

1 participant