Remove unnecessary path parameters

Author: Janther

src/slang-nodes/ConstructorAttributes.ts

@@ -32,6 +32,6 @@ export class ConstructorAttributes extends SlangNode {
   }
 
   print(path: AstPath<ConstructorAttributes>, print: PrintFunction): Doc {
-    return path.map((item) => [line, print(item)], 'items');
+    return path.map(() => [line, print(path)], 'items');
   }
 }

src/slang-nodes/FallbackFunctionAttributes.ts

@@ -32,6 +32,6 @@ export class FallbackFunctionAttributes extends SlangNode {
   }
 
   print(path: AstPath<FallbackFunctionAttributes>, print: PrintFunction): Doc {
-    return path.map((item) => [line, print(item)], 'items');
+    return path.map(() => [line, print(path)], 'items');
   }
 }

src/slang-nodes/FunctionAttributes.ts

@@ -32,6 +32,6 @@ export class FunctionAttributes extends SlangNode {
   }
 
   print(path: AstPath<FunctionAttributes>, print: PrintFunction): Doc {
-    return path.map((item) => [line, print(item)], 'items');
+    return path.map(() => [line, print(path)], 'items');
   }
 }

src/slang-nodes/FunctionTypeAttributes.ts

@@ -27,6 +27,6 @@ export class FunctionTypeAttributes extends SlangNode {
   }
 
   print(path: AstPath<FunctionTypeAttributes>, print: PrintFunction): Doc {
-    return path.map((item) => [line, print(item)], 'items');
+    return path.map(() => [line, print(path)], 'items');
   }
 }

src/slang-nodes/ModifierAttributes.ts

@@ -27,6 +27,6 @@ export class ModifierAttributes extends SlangNode {
   }
 
   print(path: AstPath<ModifierAttributes>, print: PrintFunction): Doc {
-    return path.map((item) => [line, print(item)], 'items');
+    return path.map(() => [line, print(path)], 'items');
   }
 }

src/slang-nodes/ReceiveFunctionAttributes.ts

@@ -32,6 +32,6 @@ export class ReceiveFunctionAttributes extends SlangNode {
   }
 
   print(path: AstPath<ReceiveFunctionAttributes>, print: PrintFunction): Doc {
-    return path.map((item) => [line, print(item)], 'items');
+    return path.map(() => [line, print(path)], 'items');
   }
 }

src/slang-nodes/StateVariableAttributes.ts

@@ -27,6 +27,6 @@ export class StateVariableAttributes extends SlangNode {
   }
 
   print(path: AstPath<StateVariableAttributes>, print: PrintFunction): Doc {
-    return path.map((item) => [line, print(item)], 'items');
+    return path.map(() => [line, print(path)], 'items');
   }
 }

src/slang-nodes/UnnamedFunctionAttributes.ts

@@ -32,6 +32,6 @@ export class UnnamedFunctionAttributes extends SlangNode {
   }
 
   print(path: AstPath<UnnamedFunctionAttributes>, print: PrintFunction): Doc {
-    return path.map((item) => [line, print(item)], 'items');
+    return path.map(() => [line, print(path)], 'items');
   }
 }

src/slang-printers/print-comments.ts

@@ -24,8 +24,7 @@ export function printComments(
   if (node.comments === undefined) return [];
   return joinExisting(
     line,
-    path.map((commentPath, index, comments: Comment[]) => {
-      const comment = commentPath.node;
+    path.map(({ node: comment }, index, comments: Comment[]) => {
       if (!isPrintable(comment)) {
         return '';
       }
@@ -34,7 +33,7 @@ export function printComments(
         index === comments.length - 1 ||
         comments.slice(index + 1).findIndex(isPrintable) === -1;
       return [
-        printComment(commentPath),
+        printComment(path),
         !isLast && util.isNextLineEmpty(options.originalText, locEnd(comment))
           ? hardline
           : ''

src/slang-printers/print-preserving-empty-lines.ts

@@ -16,19 +16,17 @@ export function printPreservingEmptyLines(
   options: ParserOptions<AstNode>
 ): Doc {
   return node.items.length > 0
-    ? path.map((childPath) => {
-        const node = childPath.node;
-
+    ? path.map(({ node, isFirst, isLast }) => {
         return [
           // Only attempt to prepend an empty line if `node` is not the first item
-          !childPath.isFirst &&
+          !isFirst &&
           // YulLabel adds a dedented line so we don't have to prepend a hardline.
           node.kind !== NonterminalKind.YulLabel
             ? hardline
             : '',
-          print(childPath),
+          print(path),
           // Only attempt to append an empty line if `node` is not the last item
-          !childPath.isLast &&
+          !isLast &&
           // Append an empty line if the original text already had an one after the
           // current `node`
           util.isNextLineEmpty(options.originalText, locEnd(node))