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

Fix unhelpful error message when nesting invalid. #67

Merged
merged 1 commit into from Aug 26, 2014
Merged

Fix unhelpful error message when nesting invalid. #67

merged 1 commit into from Aug 26, 2014

Conversation

untitaker
Copy link
Contributor

@coveralls
Copy link

Coverage Status

Coverage increased (+0.09%) when pulling 2956f75 on untitaker:no_parent into 1b53c81 on DiffSK:master.

@@ -1603,6 +1603,7 @@ def _parse(self, infile):
else:
self._handle_error("Section too nested",
NestingError, infile, cur_index)
continue
Copy link
Member

Choose a reason for hiding this comment

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

now I'm curious if this was ever there and then omitted

Copy link
Member

Choose a reason for hiding this comment

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

Yeah, that's strange - all the other error branches seem to have the continue statement.

robdennis added a commit that referenced this pull request Aug 26, 2014
Fix unhelpful error message when nesting invalid.
@robdennis robdennis merged commit e184b1c into DiffSK:master Aug 26, 2014
@untitaker
Copy link
Contributor Author

Thanks!

@untitaker untitaker deleted the no_parent branch August 26, 2014 00:59
@robdennis robdennis added this to the 5.0.6 milestone Aug 26, 2014
@robdennis
Copy link
Member

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.

None yet

4 participants