Require no whitespace before the '(' of a loopbody definition (#197)

StevenLooman · Dec 10, 2024 · 2489ca3 · 2489ca3
1 parent 06df102
commit 2489ca3
Show file tree

Hide file tree

Showing 2 changed files with 9 additions and 2 deletions.
diff --git a/magik-squid/src/main/java/nl/ramsolutions/sw/magik/formatting/MagikFormattingStrategy.java b/magik-squid/src/main/java/nl/ramsolutions/sw/magik/formatting/MagikFormattingStrategy.java
@@ -186,15 +186,15 @@ private boolean requireWhitespaceBefore(final Token token) {
             && (this.tokenIs(token, "<<", "^<<")))
         && !this.tokenIs(token, ".", ",", ")", "}", "]")
         && !this.tokenIs(this.lastToken, "(", "{", "[")
-        && !this.tokenIs(this.lastTextToken, "_proc"); // Except for _proc.
+        && !this.tokenIs(this.lastTextToken, "_proc", "_loopbody"); // Except for _proc/_loopbody.
   }
 
   private boolean requireNoWhitespaceBefore(final Token token) {
     final String lastTextTokenValue =
         this.lastTextToken != null ? this.lastTextToken.getOriginalValue().toLowerCase() : null;
     return !this.tokenIs(token, GenericTokenType.COMMENT)
         && (this.tokenIs(token, ")", "}", "]", ",")
-            || this.tokenIs(this.lastTextToken, "@", "(", "{", "[", "_proc")
+            || this.tokenIs(this.lastTextToken, "@", "(", "{", "[", "_proc", "_loopbody")
             || this.nodeIsSlot()
             || this.currentNode.is(MagikGrammar.ARGUMENTS)
             || this.currentNode.is(MagikGrammar.PARAMETERS)

diff --git a/magik-squid/src/test/java/nl/ramsolutions/sw/magik/formatting/FormattingWalkerTest.java b/magik-squid/src/test/java/nl/ramsolutions/sw/magik/formatting/FormattingWalkerTest.java
@@ -266,6 +266,13 @@ void testProcDefinitionParameters(final String code) {
     assertThat(edits).isEmpty();
   }
 
+  @Test
+  void testLoopbdoyDefinitionParameters() {
+    final String code = "_loopbody(x, y, z)";
+    final List<TextEdit> edits = this.getEdits(code);
+    assertThat(edits).isEmpty();
+  }
+
   // endregion
 
   // region: Indenting.