From a9c9dd5b0c7accf78f99aa1ea77b873a96e55ad5 Mon Sep 17 00:00:00 2001
From: Micha Reiser <micha@reiser.io>
Date: Tue, 11 Jul 2023 08:56:10 +0200
Subject: [PATCH] Remove obsolte TODO

---
 crates/ruff_python_formatter/src/expression/expr_subscript.rs | 4 ----
 1 file changed, 4 deletions(-)

diff --git a/crates/ruff_python_formatter/src/expression/expr_subscript.rs b/crates/ruff_python_formatter/src/expression/expr_subscript.rs
index 8e37a834317d6f..2a5e90c444aa85 100644
--- a/crates/ruff_python_formatter/src/expression/expr_subscript.rs
+++ b/crates/ruff_python_formatter/src/expression/expr_subscript.rs
@@ -90,10 +90,6 @@ impl NeedsParentheses for ExprSubscript {
         parenthesize: Parenthesize,
         context: &PyFormatContext,
     ) -> Parentheses {
-        // TODO: Issue, returns `Never` here but parentheses are necessary in return type positions.
-        // We don't have the position information available here which sux.
-        // Rename `optional_parentheses` to `parenthesize_if_breaks`
-        // Create a new optional parentheses function that does the whole magic stuff.
         match default_expression_needs_parentheses(self.into(), parenthesize, context) {
             Parentheses::Optional => Parentheses::Never,
             parentheses => parentheses,