@abaplint/runtime 2.13.49 → 2.13.50

package/build/src/statements/concatenate.js

@@ -17,19 +17,22 @@ function concatenate(input) {
         }
     }
     if (input.lines === true) {
-        const list = [];
+        let result = "";
         const tab = input.source[0];
         if (tab instanceof types_1.Table) {
-            for (const l of tab.array()) {
-                if (input.respectingBlanks !== true) {
-                    list.push(l.get().trimEnd());
-                }
-                else {
-                    list.push(l.get());
+            const array = tab.array();
+            const length = array.length;
+            const respectingBlanks = input.respectingBlanks === true;
+            const hasSeparator = sep.length > 0;
+            for (let i = 0; i < length; i++) {
+                if (hasSeparator === true && i > 0) {
+                    result += sep;
                 }
+                const value = array[i].get();
+                result += respectingBlanks === true ? value : value.trimEnd();
             }
         }
-        input.target.set(list.join(sep));
+        input.target.set(result);
     }
     else {
         let result = "";

package/build/src/statements/insert_internal.js

@@ -97,6 +97,19 @@ function insertInternal(options) {
         }
     }
     else if (options.initial === true) {
+        if (options.table instanceof types_1.HashedTable) {
+            const { value: val, subrc: subrc } = options.table.insert(options.table.getRowType());
+            if (subrc === 0) {
+                if (options.assigning) {
+                    options.assigning.assign(val);
+                }
+                if (options.referenceInto) {
+                    options.referenceInto.assign(val);
+                }
+            }
+            abap.builtin.sy.get().subrc.set(subrc);
+            return;
+        }
         let index = options.table.getArrayLength();
         if (options.index) {
             index = options.index.get() - 1;

package/build/src/statements/loop.js

@@ -31,10 +31,10 @@ function dynamicToWhere(condition, evaluate) {
     if (evaluate === undefined) {
         throw new Error("Dynamic WHERE evaluation function is not defined");
     }
-    const matches = text.matchAll(/([\w-]+)\s+(\w+)\s+([<>\w-]+)/gi);
+    const matches = text.matchAll(/([\w-]+)\s+(NOT\s+IN|\w+)\s+([<>\w-]+)/gi);
     for (const match of matches) {
         const left = match[1];
-        const comparator = match[2].toLowerCase();
+        const comparator = match[2].toLowerCase().replace(/\s+/g, " ");
         let right = "";
         //    console.dir({left, right});
         const cleft = "i." + left.toLowerCase().replace(/-/g, ".get().");
@@ -43,13 +43,21 @@ function dynamicToWhere(condition, evaluate) {
             const name = "fs_" + rightMatch[1].toLowerCase() + "_";
             right = `evaluate("${name}").get()["${rightMatch[2].toLowerCase()}"]`;
         }
+        const fieldSymbol = match[3].match(/^<(\w+)>$/);
+        if (right === "" && fieldSymbol) {
+            right = `evaluate("fs_${fieldSymbol[1].toLowerCase()}_").getPointer()`;
+        }
         if (right === "") {
-            right = `evaluate("${match[3]}")`;
+            right = `evaluate("${match[3].toLowerCase()}")`;
         }
         if (comparator === "in") {
             const cnew = `abap.compare.in(${cleft}, ${right})`;
             text = text.replace(match[0], cnew);
         }
+        else if (comparator === "not in") {
+            const cnew = `!abap.compare.in(${cleft}, ${right})`;
+            text = text.replace(match[0], cnew);
+        }
         else {
             const cnew = `abap.compare.${comparator}(${cleft}, ${right})`;
             text = text.replace(match[0], cnew);

package/build/src/types/integer.js

@@ -85,11 +85,11 @@ class Integer {
             this.set(Math.round(value.getRaw()));
         }
         else if (value instanceof hex_1.Hex || value instanceof xstring_1.XString || value instanceof hex_uint8_1.HexUInt8) {
-            let num = parseInt(value.get(), 16);
-            // handle two complement,
-            if ((value instanceof hex_1.Hex || value instanceof hex_uint8_1.HexUInt8)
-                && value.getLength() >= 4) {
-                const maxVal = Math.pow(2, value.get().length / 2 * 8);
+            const hex = value.get();
+            let num = parseInt(hex, 16);
+            // handle two complement, the first bit is the sign
+            if (hex.length >= 8) {
+                const maxVal = Math.pow(2, hex.length / 2 * 8);
                 if (num > maxVal / 2 - 1) {
                     num = num - maxVal;
                 }

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@abaplint/runtime",
-  "version": "2.13.49",
+  "version": "2.13.50",
   "description": "Transpiler - Runtime",
   "main": "build/src/index.js",
   "typings": "build/src/index.d.ts",