psf / black / 6793723653

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))* [',']
```

2950 of 3159 branches covered (0.0%)

1 of 1 new or added line in 1 file covered. (100.0%)

7019 of 7283 relevant lines covered (96.38%)

3.85 hits per line