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

Stacktrace changes #15

Closed
wants to merge 2 commits into from
Closed

Stacktrace changes #15

wants to merge 2 commits into from

Conversation

youngnh
Copy link
Contributor

@youngnh youngnh commented Apr 6, 2011

The current stacktrace middleware improperly nests the html <body> in it's <head> tag.
Additionally, it does not include the causes of an exception, only the topmost exception frame.

These commits address both of those concerns, outputting properly nested html and including all of an exception's causes.

@weavejester
Copy link
Collaborator

Could you split out the two commits into their own pull requests and branches? The 7297d06 commit looks fine and can be probably be merged in straight away, but the 37c7614 commit looks rather strange. I can't see how it can even compile. For instance, lines 58 and 59 unquote their values, but they're not inside a syntax-quote. e-parsed is defined in one function as a local variable, but used in a different function with a completely separate scope. Additionally, you reuse the same "content" id for more than one element,

@youngnh youngnh closed this Apr 7, 2011
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants