Merge pull request #705 from MarcoGorelli/remove-289-rewrite

remove pep289 rewrite
asottile · Sep 12, 2022 · 443bc1e · 443bc1e
2 parents ad5d9db + f198b1a
commit 443bc1e
Show file tree

Hide file tree

Showing 4 changed files with 0 additions and 202 deletions.
diff --git a/README.md b/README.md
@@ -58,19 +58,6 @@ Sample `.pre-commit-config.yaml`:
 ```
 
 
-### Generator expressions for some built-in functions (pep 289)
-
-```diff
--min([i for i in range(3)])
-+min(i for i in range(3))
--max([i for i in range(3)])
-+max(i for i in range(3))
--sum([i for i in range(3)])
-+sum(i for i in range(3))
--''.join([str(i) for i in range(3)])
-+''.join(str(i) for i in range(3))
-```
-
 ### Python2.7+ Format Specifiers
 
 ```diff

diff --git a/pyupgrade/_plugins/generator_expressions_pep289.py b/pyupgrade/_plugins/generator_expressions_pep289.py
diff --git a/pyupgrade/_token_helpers.py b/pyupgrade/_token_helpers.py
@@ -510,13 +510,6 @@ def find_comprehension_opening_bracket(i: int, tokens: list[Token]) -> int:
         return i
 
 
-def replace_list_comp_brackets(i: int, tokens: list[Token]) -> None:
-    start = find_comprehension_opening_bracket(i, tokens)
-    end = find_closing_bracket(tokens, start)
-    tokens[end] = Token('OP', ')')
-    tokens[start] = Token('OP', '(')
-
-
 def has_space_before(i: int, tokens: list[Token]) -> bool:
     return i >= 1 and tokens[i - 1].name in {UNIMPORTANT_WS, 'INDENT'}
 

diff --git a/tests/features/generator_expressions_pep289_test.py b/tests/features/generator_expressions_pep289_test.py