fix(fmt): avoid blank line before identifiers (#11908)

Author: 0xrusowsky

crates/fmt/src/state/mod.rs

@@ -360,7 +360,7 @@ impl State<'_, '_> {
     }
 
     fn print_ident(&mut self, ident: &ast::Ident) {
-        if self.handle_span(ident.span, false) {
+        if self.handle_span(ident.span, true) {
             return;
         }
 

crates/fmt/testdata/Repros/fmt.sol

@@ -177,6 +177,11 @@ function argListRepro(address tokenIn, uint256 amountIn, bool data) {
     );
 }
 
+// https://github.com/foundry-rs/foundry/issues/11905
+function noBlanksLinesBeforeIdentifiers() public {
+    timelockController.grantRole(keccak256("EXECUTOR_ROLE"), address(0));
+}
+
 contract NestedCallsTest is Test {
     string constant errMsg = "User provided message";
     uint256 constant maxDecimals = 77;

crates/fmt/testdata/Repros/original.sol

@@ -178,6 +178,15 @@ function argListRepro(address tokenIn, uint256 amountIn, bool data) {
     );
 }
 
+// https://github.com/foundry-rs/foundry/issues/11905
+function noBlanksLinesBeforeIdentifiers() public {
+            timelockController
+
+
+
+        .grantRole(keccak256("EXECUTOR_ROLE"), address(0));
+}
+
 contract NestedCallsTest is Test {
     string constant errMsg = "User provided message";
     uint256 constant maxDecimals = 77;

crates/fmt/testdata/Repros/sorted.fmt.sol

@@ -178,6 +178,11 @@ function argListRepro(address tokenIn, uint256 amountIn, bool data) {
     );
 }
 
+// https://github.com/foundry-rs/foundry/issues/11905
+function noBlanksLinesBeforeIdentifiers() public {
+    timelockController.grantRole(keccak256("EXECUTOR_ROLE"), address(0));
+}
+
 contract NestedCallsTest is Test {
     string constant errMsg = "User provided message";
     uint256 constant maxDecimals = 77;

crates/fmt/testdata/Repros/tab.fmt.sol

@@ -178,6 +178,11 @@ function argListRepro(address tokenIn, uint256 amountIn, bool data) {
 	);
 }
 
+// https://github.com/foundry-rs/foundry/issues/11905
+function noBlanksLinesBeforeIdentifiers() public {
+	timelockController.grantRole(keccak256("EXECUTOR_ROLE"), address(0));
+}
+
 contract NestedCallsTest is Test {
 	string constant errMsg = "User provided message";
 	uint256 constant maxDecimals = 77;