8000 bpo-33211: Change line number of decorated nodes back to def by emmatyping · Pull Request #6460 · python/cpython · GitHub
[go: up one dir, main page]
Skip to content

bpo-33211: Change line number of decor 8000 ated nodes back to def #6460

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.

Sign up for GitHub

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

Closed
wants to merge 19 commits into from
Closed

bpo-33211: Change line number of decorated nodes back to def #6460

Changes from 1 commit
Commits
Show all changes
19 commits
Select commit Hold shift + click to select a range
b60b6a6
Change line number of decorated nodes back to def
emmatyping Apr 12, 2018
ae578b0
Whitespace fixes
emmatyping Apr 12, 2018
191bd38
Fix test_dis
emmatyping Apr 12, 2018
da42832
Change fistlineno of decorated nodes in the compiler
emmatyping Apr 12, 2018
649506c
Remove unused var
emmatyping Apr 12, 2018
29b0d60
Fix indentation bug
emmatyping Apr 12, 2018
68b2228
Run patcheck
emmatyping Apr 13, 2018
09d8ab2
Merge remote tracking branch into deflineno
emmatyping Apr 22, 2018
df9fbe2
Change tests back to original state
emmatyping Apr 23, 2018
9840f84
Fix line number in compiler
emmatyping Apr 23, 2018
0842c20
Use unreachable macro
emmatyping Apr 23, 2018
7f4fcdf
Fix silly indentation mistake
emmatyping Apr 23, 2018
e4e7f1a
Test ast line numbers for decorated nodes
emmatyping Apr 24, 2018
a8e747d
Fix true order tests for decorators
emmatyping Apr 24, 2018
3df5862
Fix bad indentation
emmatyping Apr 24, 2018
44a3cd9
Respond to review
emmatyping Apr 24, 2018
afd3b7e
Fix regression with inner decorated nodes
emmatyping Apr 24, 2018
bdc9942
Remove redundant line number setting
emmatyping May 7, 2018
914d0a0
Fix style in compile.c
emmatyping May 7, 2018
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
Whitespace fixes
  • Loading branch information
@emmatyping
emmatyping committed Apr 12, 2018
commit ae578b04d247c7555dc8351b569d22e32656bcef
6 changes: 3 additions & 3 deletions Python/ast.c
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -1686,11 +1686,11 @@ ast_for_decorated(struct compiling *c, const node *n)
TYPE(CHILD(n, 1)) == classdef);

if (TYPE(CHILD(n, 1)) == funcdef) {
return ast_for_funcdef(c, CHILD(n, 1), decorator_seq);
return ast_for_funcdef(c, CHILD(n, 1), decorator_seq);
} else if (TYPE(CHILD(n, 1)) == classdef) {
Copy link
Member

Choose a reason for hiding this comment

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

Actually now you can remove all elses.

Copy link
Member Author

Choose a reason for hiding this comment

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

You'd think so, sadly MSVC warns that not all code paths return if there isn't an else.

Copy link
Member

Choose a reason for hiding this comment

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

Even with Py_UNREACHABLE()?

Copy link
Member Author

Choose a reason for hiding this comment

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

No, with the Py_UNREACHABLE() it is happy. Do you want something like this?

    if (TYPE(CHILD(n, 1)) == funcdef) {
        return ast_for_funcdef(c, CHILD(n, 1), decorator_seq);
    } else if (TYPE(CHILD(n, 1)) == classdef) {
        return ast_for_classdef(c, CHILD(n, 1), decorator_seq);
    } else if (TYPE(CHILD(n, 1)) == async_funcdef) {
        return ast_for_async_funcdef(c, CHILD(n, 1), decorator_seq);
    }
    Py_UNREACHABLE();

Copy link
Member

Choose a reason for hiding this comment

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

I suggest to remove other elses too.

return ast_for_classdef(c, CHILD(n, 1), decorator_seq);
return ast_for_classdef(c, CHILD(n, 1), decorator_seq);
} else if (TYPE(CHILD(n, 1)) == async_funcdef) {
return ast_for_async_funcdef(c, CHILD(n, 1), decorator_seq);
return ast_for_async_funcdef(c, CHILD(n, 1), decorator_seq);
}
}

Expand Down
0