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

Crash when property accidentally defined twice #36

Closed
dirkschulze opened this issue Sep 14, 2013 · 2 comments
Closed

Crash when property accidentally defined twice #36

dirkschulze opened this issue Sep 14, 2013 · 2 comments

Comments

@dirkschulze
Copy link
Contributor

If one uses <pre class='propdef'> twice with the same property name for Name: twice, bikeshed crashes with:

Traceback (most recent call last):
File "/Users/dschulze/Downloads/bikeshed/bikeshed.py", line 1557, in
main()
File "/Users/dschulze/Downloads/bikeshed/bikeshed.py", line 103, in main
config.doc.preprocess()
File "/Users/dschulze/Downloads/bikeshed/bikeshed.py", line 1161, in preprocess
processAutolinks(self)
File "/Users/dschulze/Downloads/bikeshed/bikeshed.py", line 884, in processAutolinks
error=(text not in doc.ignoredTerms))
File "/Users/dschulze/Downloads/bikeshed/lib/ReferenceManager.py", line 116, in getRef
refs[0]['type'],
UnboundLocalError: local variable 'refs' referenced before assignment

@dirkschulze
Copy link
Contributor Author

Same with definitions of keywords. Even if <dl> list has different dfn-fors.

@tabatkins
Copy link
Collaborator

Fixed - copypaste error from when I moved some code around. This should have died at an earlier point, though, when it detected the duplicate propdef. I'll have to look into that separately.

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

No branches or pull requests

2 participants