Don't drop "?" on nullable empty record types.

Fix #1224.
dart-lang · May 25, 2023 · 873fb1d · 873fb1d
1 parent a36eb20
commit 873fb1d
Show file tree

Hide file tree

Showing 3 changed files with 7 additions and 1 deletion.
diff --git a/CHANGELOG.md b/CHANGELOG.md
@@ -7,6 +7,7 @@
 * Don't split before `.` following a record literal (#1213).
 * Don't force split on a line comment before a switch expression case (#1215).
 * Require `package:analyzer` `^5.12.0`.
+* Preserve `?` on nullable empty record types (#1224).
 
 # 2.3.1
 

diff --git a/lib/src/source_visitor.dart b/lib/src/source_visitor.dart
@@ -2505,6 +2505,7 @@ class SourceVisitor extends ThrowingAstVisitor {
       if (node.rightParenthesis.precedingComments != null) soloZeroSplit();
 
       token(node.rightParenthesis);
+      token(node.question);
       return;
     }
 

diff --git a/test/whitespace/record_types.unit b/test/whitespace/record_types.unit
@@ -47,4 +47,8 @@
 >>> nullable record type
 (  int  ,   bool  )   ?  x;
 <<<
-(int, bool)? x;
+(int, bool)? x;
+>>> empty nullable record type
+(  )  ?   x;
+<<<
+()? x;