psf / black / 6793723653 / 1

96%

master: 96%

push

github

Remove redundant condition from `has_magic_trailing_comma` (#4023)

The second `if` cannot be true at its execution point, because it is
already covered by the first `if`. The condition
`comma.parent.type == syms.subscriptlist` always holds if
`closing.parent.type == syms.trailer` holds, because `subscriptlist`
only appears inside `trailer` in the grammar:

```
trailer: '(' [arglist] ')' | '[' subscriptlist ']' | '.' NAME
subscriptlist: (subscript|star_expr) (',' (subscript|star_expr))* [',']
```

3780 of 4124 branches covered (0.0%)

7011 of 7283 relevant lines covered (96.27%)

0.96 hits per line