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
Additional refactors based on the PR #2206 #2379
Conversation
|
||
return True | ||
def _extracted_from_maybe_make_parens_invisible_in_atom_28(node, first, last, parent): |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
What does 28 mean here?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Sorry, the number 28
is just a filler for the help-functions, but actually not needed
mo = re.match(r"^#define\s+(\w+)\s+(\d+)$", line) | ||
if not mo and line.strip(): | ||
print("%s(%s): can't parse %s" % (filename, lineno, line.strip())) | ||
print("%s(%s): can't parse %s" % (filename, lineno+1, line.strip())) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
You need either this or the start=1 but not both.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yes, you are right!
Additional refactors based on the PR #2206