small tweak with the fact that empty string has 2 characters and the minimised variable name has only 1

Janther · Janther · commit 2eed983285e4 · 2026-04-13T11:41:11.000-04:00
diff --git a/src/slang-nodes/AssemblyStatement.ts b/src/slang-nodes/AssemblyStatement.ts
@@ -41,8 +41,8 @@ export class AssemblyStatement extends SlangNode {
     const flagsDoc = print('flags');
     return [
       'assembly ',
-      labelDoc ? [labelDoc, ' '] : '',
-      flagsDoc ? [flagsDoc, ' '] : '',
+      labelDoc ? [labelDoc, ' '] : labelDoc,
+      flagsDoc ? [flagsDoc, ' '] : flagsDoc,
       print('body')
     ];
   }
diff --git a/src/slang-nodes/DecimalNumberExpression.ts b/src/slang-nodes/DecimalNumberExpression.ts
@@ -26,6 +26,6 @@ export class DecimalNumberExpression extends SlangNode {
 
   print(print: PrintFunction): Doc {
     const unitDoc = print('unit');
-    return [this.literal, unitDoc ? [' ', unitDoc] : ''];
+    return [this.literal, unitDoc ? [' ', unitDoc] : unitDoc];
   }
 }
diff --git a/src/slang-nodes/ErrorParameter.ts b/src/slang-nodes/ErrorParameter.ts
@@ -35,6 +35,6 @@ export class ErrorParameter extends SlangNode {
 
   print(print: PrintFunction): Doc {
     const nameDoc = print('name');
-    return [print('typeName'), nameDoc ? [' ', nameDoc] : ''];
+    return [print('typeName'), nameDoc ? [' ', nameDoc] : nameDoc];
   }
 }
diff --git a/src/slang-nodes/EventParameter.ts b/src/slang-nodes/EventParameter.ts
@@ -41,7 +41,7 @@ export class EventParameter extends SlangNode {
     return [
       print('typeName'),
       this.indexedKeyword ? ` ${this.indexedKeyword}` : '',
-      nameDoc ? [' ', nameDoc] : ''
+      nameDoc ? [' ', nameDoc] : nameDoc
     ];
   }
 }
diff --git a/src/slang-nodes/HexNumberExpression.ts b/src/slang-nodes/HexNumberExpression.ts
@@ -26,6 +26,6 @@ export class HexNumberExpression extends SlangNode {
 
   print(print: PrintFunction): Doc {
     const unitDoc = print('unit');
-    return [this.literal, unitDoc ? [' ', unitDoc] : ''];
+    return [this.literal, unitDoc ? [' ', unitDoc] : unitDoc];
   }
 }
diff --git a/src/slang-nodes/MappingKey.ts b/src/slang-nodes/MappingKey.ts
@@ -28,6 +28,6 @@ export class MappingKey extends SlangNode {
 
   print(print: PrintFunction): Doc {
     const nameDoc = print('name');
-    return [print('keyType'), nameDoc ? [' ', nameDoc] : ''];
+    return [print('keyType'), nameDoc ? [' ', nameDoc] : nameDoc];
   }
 }
diff --git a/src/slang-nodes/MappingValue.ts b/src/slang-nodes/MappingValue.ts
@@ -35,6 +35,6 @@ export class MappingValue extends SlangNode {
 
   print(print: PrintFunction): Doc {
     const nameDoc = print('name');
-    return [print('typeName'), nameDoc ? [' ', nameDoc] : ''];
+    return [print('typeName'), nameDoc ? [' ', nameDoc] : nameDoc];
   }
 }
diff --git a/src/slang-nodes/Parameter.ts b/src/slang-nodes/Parameter.ts
@@ -50,8 +50,8 @@ export class Parameter extends SlangNode {
     const nameDoc = print('name');
     return group([
       print('typeName'),
-      storageLocationDoc ? [' ', storageLocationDoc] : '',
-      nameDoc ? [' ', nameDoc] : ''
+      storageLocationDoc ? [' ', storageLocationDoc] : storageLocationDoc,
+      nameDoc ? [' ', nameDoc] : nameDoc
     ]);
   }
 }
diff --git a/src/slang-nodes/TryStatement.ts b/src/slang-nodes/TryStatement.ts
@@ -58,7 +58,7 @@ export class TryStatement extends SlangNode {
         firstSeparator: line
       }),
       [
-        returnsDoc ? [returnsDoc, ' '] : '',
+        returnsDoc ? [returnsDoc, ' '] : returnsDoc,
         print('body'),
         ' ',
         print('catchClauses')
diff --git a/src/slang-nodes/TypedTupleMember.ts b/src/slang-nodes/TypedTupleMember.ts
@@ -44,7 +44,7 @@ export class TypedTupleMember extends SlangNode {
     const storageLocationDoc = print('storageLocation');
     return [
       print('typeName'),
-      storageLocationDoc ? [' ', storageLocationDoc] : '',
+      storageLocationDoc ? [' ', storageLocationDoc] : storageLocationDoc,
       ' ',
       print('name')
     ];
diff --git a/src/slang-nodes/UntypedTupleMember.ts b/src/slang-nodes/UntypedTupleMember.ts
@@ -30,6 +30,9 @@ export class UntypedTupleMember extends SlangNode {
 
   print(print: PrintFunction): Doc {
     const storageLocationDoc = print('storageLocation');
-    return [storageLocationDoc ? [storageLocationDoc, ' '] : '', print('name')];
+    return [
+      storageLocationDoc ? [storageLocationDoc, ' '] : storageLocationDoc,
+      print('name')
+    ];
   }
 }
diff --git a/src/slang-nodes/YulVariableDeclarationStatement.ts b/src/slang-nodes/YulVariableDeclarationStatement.ts
@@ -36,6 +36,6 @@ export class YulVariableDeclarationStatement extends SlangNode {
 
   print(print: PrintFunction): Doc {
     const valueDoc = print('value');
-    return ['let', print('variables'), valueDoc ? [' ', valueDoc] : ''];
+    return ['let', print('variables'), valueDoc ? [' ', valueDoc] : valueDoc];
   }
 }
diff --git a/src/slang-printers/print-function.ts b/src/slang-printers/print-function.ts
@@ -20,7 +20,7 @@ export function printFunction(
       indent(
         group([
           print('attributes'),
-          returnsDoc ? [line, returnsDoc] : '',
+          returnsDoc ? [line, returnsDoc] : returnsDoc,
           (node as FunctionWithBody).body?.kind === NonterminalKind.Block
             ? dedent(line)
             : ''

Original file line number	Diff line number	Diff line change
`@@ -26,6 +26,6 @@ export class DecimalNumberExpression extends SlangNode {`
`26`	`26`
`27`	`27`	`print(print: PrintFunction): Doc {`
`28`	`28`	`const unitDoc = print('unit');`
`29`		`- return [this.literal, unitDoc ? [' ', unitDoc] : ''];`
	`29`	`+ return [this.literal, unitDoc ? [' ', unitDoc] : unitDoc];`
`30`	`30`	`}`
`31`	`31`	`}`
Original file line number	Diff line number	Diff line change
`@@ -35,6 +35,6 @@ export class ErrorParameter extends SlangNode {`
`35`	`35`
`36`	`36`	`print(print: PrintFunction): Doc {`
`37`	`37`	`const nameDoc = print('name');`
`38`		`- return [print('typeName'), nameDoc ? [' ', nameDoc] : ''];`
	`38`	`+ return [print('typeName'), nameDoc ? [' ', nameDoc] : nameDoc];`
`39`	`39`	`}`
`40`	`40`	`}`
Original file line number	Diff line number	Diff line change
`@@ -41,7 +41,7 @@ export class EventParameter extends SlangNode {`
`41`	`41`	`return [`
`42`	`42`	`print('typeName'),`
`43`	`43`	this.indexedKeyword ? ` ${this.indexedKeyword}` : '',
`44`		`- nameDoc ? [' ', nameDoc] : ''`
	`44`	`+ nameDoc ? [' ', nameDoc] : nameDoc`
`45`	`45`	`];`
`46`	`46`	`}`
`47`	`47`	`}`
Original file line number	Diff line number	Diff line change
`@@ -26,6 +26,6 @@ export class HexNumberExpression extends SlangNode {`
`26`	`26`
`27`	`27`	`print(print: PrintFunction): Doc {`
`28`	`28`	`const unitDoc = print('unit');`
`29`		`- return [this.literal, unitDoc ? [' ', unitDoc] : ''];`
	`29`	`+ return [this.literal, unitDoc ? [' ', unitDoc] : unitDoc];`
`30`	`30`	`}`
`31`	`31`	`}`
Original file line number	Diff line number	Diff line change
`@@ -28,6 +28,6 @@ export class MappingKey extends SlangNode {`
`28`	`28`
`29`	`29`	`print(print: PrintFunction): Doc {`
`30`	`30`	`const nameDoc = print('name');`
`31`		`- return [print('keyType'), nameDoc ? [' ', nameDoc] : ''];`
	`31`	`+ return [print('keyType'), nameDoc ? [' ', nameDoc] : nameDoc];`
`32`	`32`	`}`
`33`	`33`	`}`
Original file line number	Diff line number	Diff line change
`@@ -35,6 +35,6 @@ export class MappingValue extends SlangNode {`
`35`	`35`
`36`	`36`	`print(print: PrintFunction): Doc {`
`37`	`37`	`const nameDoc = print('name');`
`38`		`- return [print('typeName'), nameDoc ? [' ', nameDoc] : ''];`
	`38`	`+ return [print('typeName'), nameDoc ? [' ', nameDoc] : nameDoc];`
`39`	`39`	`}`
`40`	`40`	`}`
Original file line number	Diff line number	Diff line change
`@@ -50,8 +50,8 @@ export class Parameter extends SlangNode {`
`50`	`50`	`const nameDoc = print('name');`
`51`	`51`	`return group([`
`52`	`52`	`print('typeName'),`
`53`		`- storageLocationDoc ? [' ', storageLocationDoc] : '',`
`54`		`- nameDoc ? [' ', nameDoc] : ''`
	`53`	`+ storageLocationDoc ? [' ', storageLocationDoc] : storageLocationDoc,`
	`54`	`+ nameDoc ? [' ', nameDoc] : nameDoc`
`55`	`55`	`]);`
`56`	`56`	`}`
`57`	`57`	`}`
Original file line number	Diff line number	Diff line change
`@@ -58,7 +58,7 @@ export class TryStatement extends SlangNode {`
`58`	`58`	`firstSeparator: line`
`59`	`59`	`}),`
`60`	`60`	`[`
`61`		`- returnsDoc ? [returnsDoc, ' '] : '',`
	`61`	`+ returnsDoc ? [returnsDoc, ' '] : returnsDoc,`
`62`	`62`	`print('body'),`
`63`	`63`	`' ',`
`64`	`64`	`print('catchClauses')`