@abaplint/runtime 2.8.3 → 2.8.5

package/build/src/builtin/abs.js

@@ -17,7 +17,8 @@ function abs(input) {
     else {
         num_in = parseFloat(input.val.get().toString());
     }
-    return Math.abs(num_in);
+    const ret = Math.abs(num_in);
+    return ret;
 }
 exports.abs = abs;
 //# sourceMappingURL=abs.js.map

package/build/src/operators/mod.js

@@ -7,8 +7,11 @@ function mod(left, right) {
     if (left instanceof types_1.Integer8 || right instanceof types_1.Integer8) {
         const l = left instanceof types_1.Integer8 ? left.get() : BigInt((0, _parse_1.parse)(left));
         const r = right instanceof types_1.Integer8 ? right.get() : BigInt((0, _parse_1.parse)(right));
-        // todo, is this correct?
-        return new types_1.Integer8().set(l % r);
+        let val = ((l % r) + r) % r;
+        if (val < 0) {
+            val = val * -1n;
+        }
+        return new types_1.Integer8().set(val);
     }
     const l = (0, _parse_1.parse)(left);
     const r = (0, _parse_1.parse)(right);

package/build/src/types/hex.js

@@ -22,7 +22,11 @@ class Hex {
         else if (typeof value === "number") {
             const maxVal = Math.pow(2, this.length * 8);
             if (value < 0) {
-                this.value = Math.round(value + maxVal).toString(16);
+                let hex = Math.round(value + 0x100000000).toString(16).toUpperCase();
+                if (hex.length > this.length * 2) {
+                    hex = hex.substring(hex.length - this.length * 2);
+                }
+                this.value = hex;
             }
             else if (value >= maxVal) {
                 const sub = value % maxVal;

package/package.json

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