@abaplint/transpiler 2.13.98 → 2.13.99

package/build/src/traversal.d.ts

@@ -15,6 +15,10 @@ export declare class Traversal {
     private readonly doOrWhileIndexBackups;
     private readonly statementsInsideLoop;
     private readonly enclosingDoOrWhile;
+    /** DATA, CONSTANTS and FIELD-SYMBOLS nested in blocks, already declared in front of the block */
+    private readonly declaredEarly;
+    private emittingEarly;
+    private blockDepth;
     readonly reg: abaplint.IRegistry;
     readonly options: ITranspilerOptions | undefined;
     constructor(spaghetti: abaplint.ISpaghettiScope, file: abaplint.ABAPFile, obj: abaplint.ABAPObject, reg: abaplint.IRegistry, options?: ITranspilerOptions);
@@ -106,6 +110,15 @@ export declare class Traversal {
     buildInternalName(name: string, def: abaplint.IClassDefinition | abaplint.IInterfaceDefinition | undefined): string;
     lookupClassOrInterface(name: string | undefined, token: abaplint.Token | undefined, directGlobal?: boolean): string;
     buildPrefix(): string;
+    /** In ABAP, DATA, CONSTANTS and FIELD-SYMBOLS are visible in the whole method/form/program, but a
+     * javascript "let" inside the block of an IF, LOOP, TRY etc. is not. So before each block on
+     * method/form/program level the declarations nested in it are output, in source order, and
+     * skipped inside the block. They then exist after the block even if the branch did not run,
+     * and are initialized once, also when the block is a loop */
+    private traverseProcedureLevel;
+    private declareNestedEarly;
+    private static isDeclaration;
+    private static findNestedDeclarations;
     protected traverseStructure(node: abaplint.Nodes.StructureNode): Chunk;
     protected traverseStatement(node: abaplint.Nodes.StatementNode): Chunk;
     protected traverseExpression(node: abaplint.Nodes.ExpressionNode): Chunk;

package/build/src/traversal.js

@@ -88,6 +88,10 @@ class Traversal {
     doOrWhileIndexBackups = new Map();
     statementsInsideLoop = new WeakSet();
     enclosingDoOrWhile = new WeakMap();
+    /** DATA, CONSTANTS and FIELD-SYMBOLS nested in blocks, already declared in front of the block */
+    declaredEarly = new WeakSet();
+    emittingEarly = false;
+    blockDepth = 0;
     reg;
     options;
     constructor(spaghetti, file, obj, reg, options) {
@@ -926,17 +930,15 @@ this.INTERNAL_ID = abap.internalIdCounter++;\n`;
             }
             else if (c.get() instanceof abaplint.Expressions.AttributeName
                 && context instanceof abaplint.BasicTypes.ObjectReferenceType) {
-                const id = context.getIdentifier();
-                if (id instanceof abaplint.Types.ClassDefinition || id instanceof abaplint.Types.InterfaceDefinition) {
-                    const concat = c.concatTokens();
-                    if (concat.includes("~")) {
-                        const [iname, aname] = concat.split("~");
-                        const intf = this.findInterfaceDefinition(iname, scope);
-                        context = intf?.getAttributes().findByName(aname)?.getType();
-                    }
-                    else {
-                        context = id.getAttributes().findByName(concat)?.getType();
-                    }
+                const concat = c.concatTokens();
+                if (concat.includes("~")) {
+                    const [iname, aname] = concat.split("~");
+                    const intf = this.findInterfaceDefinition(iname, scope);
+                    context = intf?.getAttributes().findByName(aname)?.getType();
+                }
+                else {
+                    // Global references can carry an identifier rather than a class definition.
+                    context = this.narrowContextAttribute(context, concat, scope);
                 }
             }
             else if (c.get() instanceof abaplint.Expressions.AttributeName
@@ -950,7 +952,7 @@ this.INTERNAL_ID = abap.internalIdCounter++;\n`;
         return context;
     }
     isInsideLoop(node) {
-        return this.statementsInsideLoop.has(node);
+        return this.statementsInsideLoop.has(node) && !this.declaredEarly.has(node);
     }
     isInsideDoOrWhile(node) {
         return this.findCurrentDoOrWhileIndexBackup(node) !== undefined;
@@ -1045,7 +1047,87 @@ this.INTERNAL_ID = abap.internalIdCounter++;\n`;
         return this.obj.getType() + "-" + this.obj.getName() + "-";
     }
     ////////////////////////////
+    /** In ABAP, DATA, CONSTANTS and FIELD-SYMBOLS are visible in the whole method/form/program, but a
+     * javascript "let" inside the block of an IF, LOOP, TRY etc. is not. So before each block on
+     * method/form/program level the declarations nested in it are output, in source order, and
+     * skipped inside the block. They then exist after the block even if the branch did not run,
+     * and are initialized once, also when the block is a loop */
+    traverseProcedureLevel(node) {
+        const ret = new chunk_1.Chunk();
+        for (const c of node.getChildren()) {
+            if (c instanceof abaplint.Nodes.StructureNode && Traversal.isDeclaration(c) === false) {
+                ret.appendChunk(this.declareNestedEarly(c));
+                this.blockDepth++;
+                try {
+                    ret.appendChunk(this.traverseStructure(c));
+                }
+                finally {
+                    this.blockDepth--;
+                }
+            }
+            else {
+                ret.appendChunk(this.traverse(c));
+            }
+        }
+        return ret;
+    }
+    declareNestedEarly(block) {
+        const found = [];
+        Traversal.findNestedDeclarations(block, found);
+        for (const node of found) {
+            this.declaredEarly.add(node);
+            if (node instanceof abaplint.Nodes.StructureNode) {
+                for (const statement of node.findAllStatementNodes()) {
+                    this.declaredEarly.add(statement);
+                }
+            }
+        }
+        const ret = new chunk_1.Chunk();
+        this.emittingEarly = true;
+        try {
+            for (const node of found) {
+                ret.appendChunk(this.traverse(node));
+            }
+        }
+        finally {
+            this.emittingEarly = false;
+        }
+        return ret;
+    }
+    static isDeclaration(node) {
+        const get = node.get();
+        return get instanceof abaplint.Structures.Data
+            || get instanceof abaplint.Structures.Constants
+            || get instanceof abaplint.Structures.Types;
+    }
+    static findNestedDeclarations(node, found) {
+        for (const c of node.getChildren()) {
+            if (c instanceof abaplint.Nodes.StatementNode) {
+                const get = c.get();
+                if (get instanceof abaplint.Statements.Data
+                    || get instanceof abaplint.Statements.Constant
+                    || get instanceof abaplint.Statements.FieldSymbol) {
+                    found.push(c);
+                }
+            }
+            else if (c instanceof abaplint.Nodes.StructureNode) {
+                const get = c.get();
+                if (get instanceof abaplint.Structures.Data || get instanceof abaplint.Structures.Constants) {
+                    found.push(c);
+                }
+                else {
+                    Traversal.findNestedDeclarations(c, found);
+                }
+            }
+        }
+    }
     traverseStructure(node) {
+        if (this.declaredEarly.has(node) && this.emittingEarly === false) {
+            return new chunk_1.Chunk();
+        }
+        else if (this.blockDepth === 0 && node.get() instanceof abaplint.Structures.Normal) {
+            return this.traverseProcedureLevel(node);
+        }
         const list = StructureTranspilers;
         const ret = new chunk_1.Chunk();
         const search = node.get().constructor.name + "Transpiler";
@@ -1068,6 +1150,9 @@ this.INTERNAL_ID = abap.internalIdCounter++;\n`;
         return ret;
     }
     traverseStatement(node) {
+        if (this.declaredEarly.has(node) && this.emittingEarly === false) {
+            return new chunk_1.Chunk();
+        }
         const list = StatementTranspilers;
         const search = node.get().constructor.name + "Transpiler";
         if (list[search]) {

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@abaplint/transpiler",
-  "version": "2.13.98",
+  "version": "2.13.99",
   "description": "Transpiler",
   "main": "build/src/index.js",
   "typings": "build/src/index.d.ts",
@@ -29,7 +29,7 @@
   "author": "abaplint",
   "license": "MIT",
   "dependencies": {
-    "@abaplint/core": "^2.120.66",
+    "@abaplint/core": "^2.120.68",
     "source-map": "^0.8.0"
   },
   "devDependencies": {