fix(fmt): emit stmt grouping (foundry-rs#4430)

fmt/src/formatter.rs

@@ -2276,12 +2276,9 @@ impl<'a, W: Write> Visitor for Formatter<'a, W> {
 
     fn visit_emit(&mut self, loc: Loc, event: &mut Expression) -> Result<()> {
         return_source_if_disabled!(self, loc);
-        self.grouped(|fmt| {
-            write_chunk!(fmt, loc.start(), "emit")?;
-            event.visit(fmt)?;
-            fmt.write_semicolon()?;
-            Ok(())
-        })?;
+        write_chunk!(self, loc.start(), "emit")?;
+        event.visit(self)?;
+        self.write_semicolon()?;
         Ok(())
     }
 
@@ -3674,4 +3671,5 @@ mod tests {
     test_directory! { PragmaDirective }
     test_directory! { Annotation }
     test_directory! { MappingType }
+    test_directory! { EmitStatement }
 }

fmt/testdata/EmitStatement/fmt.sol

@@ -0,0 +1,31 @@
+// config: line_length = 80
+event NewEvent(
+    address beneficiary, uint256 index, uint64 timestamp, uint64 endTimestamp
+);
+
+function emitEvent() {
+    emit NewEvent(
+        beneficiary,
+        _vestingBeneficiaries.length - 1,
+        uint64(block.timestamp),
+        endTimestamp
+    );
+
+    emit NewEvent(
+        /* beneficiary */
+        beneficiary,
+        /* index */
+        _vestingBeneficiaries.length - 1,
+        /* timestamp */
+        uint64(block.timestamp),
+        /* end timestamp */
+        endTimestamp
+    );
+
+    emit NewEvent(
+        beneficiary, // beneficiary
+        _vestingBeneficiaries.length - 1, // index
+        uint64(block.timestamp), // timestamp
+        endTimestamp // end timestamp
+    );
+}

fmt/testdata/EmitStatement/original.sol

@@ -0,0 +1,24 @@
+event NewEvent(address beneficiary, uint256 index, uint64 timestamp, uint64 endTimestamp);
+
+function emitEvent() {
+    emit NewEvent(
+            beneficiary,
+            _vestingBeneficiaries.length - 1,
+            uint64(block.timestamp),
+            endTimestamp
+            );
+
+    emit
+        NewEvent(
+            /* beneficiary */ beneficiary,
+            /* index */ _vestingBeneficiaries.length - 1,
+        /* timestamp */   uint64(block.timestamp),
+    /* end timestamp */ endTimestamp);
+
+    emit NewEvent(
+        beneficiary, // beneficiary
+      _vestingBeneficiaries.length - 1, // index
+   uint64(block.timestamp), // timestamp
+     endTimestamp // end timestamp
+    ); 
+}