Merge pull request #4340 from harupy/fix-locations-of-parethesized-expressions

Fix the start and end locations of `Tuple`

Author: DimitrisJim

parser/python.lalrpop

@@ -1089,7 +1089,7 @@ AtomAs<Goal>: Goal = {
             node: ast::ExprKind::ListComp { elt: Box::new(elt), generators }
         }.into()
     },
-    "(" <location:@L> <items:OneOrMore<TestOrStarNamedExprOrWithitem>> <trailing_comma:","?> <end_location:@R> ")" =>? {
+    <location:@L> "(" <items:OneOrMore<TestOrStarNamedExprOrWithitem>> <trailing_comma:","?> ")" <end_location:@R> =>? {
         if items.len() == 1 && items[0].1.is_none() && trailing_comma.is_none() {
             match items[0].0.node {
                 ast::ExprKind::Starred { .. } => {

parser/src/snapshots/rustpython_parser__context__tests__assign_attribute.snap

@@ -55,12 +55,12 @@ expression: parse_ast
             value: Located {
                 location: Location {
                     row: 1,
-                    column: 7,
+                    column: 6,
                 },
                 end_location: Some(
                     Location {
                         row: 1,
-                        column: 14,
+                        column: 15,
                     },
                 ),
                 custom: (),

parser/src/snapshots/rustpython_parser__context__tests__assign_for.snap

@@ -36,12 +36,12 @@ expression: parse_ast
             iter: Located {
                 location: Location {
                     row: 1,
-                    column: 10,
+                    column: 9,
                 },
                 end_location: Some(
                     Location {
                         row: 1,
-                        column: 17,
+                        column: 18,
                     },
                 ),
                 custom: (),

parser/src/snapshots/rustpython_parser__context__tests__assign_list.snap

@@ -73,12 +73,12 @@ expression: parse_ast
             value: Located {
                 location: Location {
                     row: 1,
-                    column: 10,
+                    column: 9,
                 },
                 end_location: Some(
                     Location {
                         row: 1,
-                        column: 17,
+                        column: 18,
                     },
                 ),
                 custom: (),

parser/src/snapshots/rustpython_parser__context__tests__assign_list_comp.snap

@@ -87,12 +87,12 @@ expression: parse_ast
                             iter: Located {
                                 location: Location {
                                     row: 1,
-                                    column: 17,
+                                    column: 16,
                                 },
                                 end_location: Some(
                                     Location {
                                         row: 1,
-                                        column: 24,
+                                        column: 25,
                                     },
                                 ),
                                 custom: (),

parser/src/snapshots/rustpython_parser__context__tests__assign_name.snap

@@ -38,12 +38,12 @@ expression: parse_ast
             value: Located {
                 location: Location {
                     row: 1,
-                    column: 5,
+                    column: 4,
                 },
                 end_location: Some(
                     Location {
                         row: 1,
-                        column: 12,
+                        column: 13,
                     },
                 ),
                 custom: (),

parser/src/snapshots/rustpython_parser__context__tests__assign_set_comp.snap

@@ -87,12 +87,12 @@ expression: parse_ast
                             iter: Located {
                                 location: Location {
                                     row: 1,
-                                    column: 17,
+                                    column: 16,
                                 },
                                 end_location: Some(
                                     Location {
                                         row: 1,
-                                        column: 24,
+                                        column: 25,
                                     },
                                 ),
                                 custom: (),

parser/src/snapshots/rustpython_parser__context__tests__assign_starred.snap

@@ -20,12 +20,12 @@ expression: parse_ast
                 Located {
                     location: Location {
                         row: 1,
-                        column: 1,
+                        column: 0,
                     },
                     end_location: Some(
                         Location {
                             row: 1,
-                            column: 6,
+                            column: 7,
                         },
                     ),
                     custom: (),
@@ -89,12 +89,12 @@ expression: parse_ast
             value: Located {
                 location: Location {
                     row: 1,
-                    column: 11,
+                    column: 10,
                 },
                 end_location: Some(
                     Location {
                         row: 1,
-                        column: 18,
+                        column: 19,
                     },
                 ),
                 custom: (),

parser/src/snapshots/rustpython_parser__context__tests__assign_subscript.snap

@@ -71,12 +71,12 @@ expression: parse_ast
             value: Located {
                 location: Location {
                     row: 1,
-                    column: 8,
+                    column: 7,
                 },
                 end_location: Some(
                     Location {
                         row: 1,
-                        column: 15,
+                        column: 16,
                     },
                 ),
                 custom: (),

parser/src/snapshots/rustpython_parser__context__tests__assign_tuple.snap

@@ -20,12 +20,12 @@ expression: parse_ast
                 Located {
                     location: Location {
                         row: 1,
-                        column: 1,
+                        column: 0,
                     },
                     end_location: Some(
                         Location {
                             row: 1,
-                            column: 5,
+                            column: 6,
                         },
                     ),
                     custom: (),
@@ -73,12 +73,12 @@ expression: parse_ast
             value: Located {
                 location: Location {
                     row: 1,
-                    column: 10,
+                    column: 9,
                 },
                 end_location: Some(
                     Location {
                         row: 1,
-                        column: 17,
+                        column: 18,
                     },
                 ),
                 custom: (),