@abaplint/cli 2.120.70 → 2.120.72

package/build/cli.js

@@ -353,7 +353,7 @@ class Lexer {
             else if (this.m === ModeTemplate
                 && this.buffer.length() > 1
                 && (current === CH_PIPE || current === CH_LBRACE)
-                && (stream.prevChar() !== CH_BACKSLASH || (stream.prevPrevChar() === CH_BACKSLASH && stream.prevChar() === CH_BACKSLASH))) {
+                && stream.countPrevious(CH_BACKSLASH) % 2 === 0) {
                 // end of template
                 if (current === CH_LBRACE || AFTER_LITERAL.has(ahead)) {
                     this.add();
@@ -507,6 +507,14 @@ class LexerStream {
         }
         return this.raw.charCodeAt(o);
     }
+    /** number of consecutive occurrences of the character directly before the current one */
+    countPrevious(code) {
+        let count = 0;
+        for (let o = this.offset - 1; o >= 0 && this.raw.charCodeAt(o) === code; o--) {
+            count++;
+        }
+        return count;
+    }
     currentChar() {
         if (this.offset < 0) {
             return NL; // simulate newline at start of file to handle star(*) comments
@@ -10389,7 +10397,7 @@ class StringTemplateFormatting extends combi_1.Expression {
         const alpha = (0, combi_1.ver)(version_1.Release.v740sp02, (0, combi_1.seq)("ALPHA =", alphaOptions), { also: combi_1.AlsoIn.OpenABAP });
         const xsd = (0, combi_1.ver)(version_1.Release.v740sp02, (0, combi_1.seq)("XSD =", zeroXSDOptions));
         const country = (0, combi_1.seq)("COUNTRY =", _1.Source);
-        const formatting = (0, combi_1.altPrio)((0, combi_1.seq)("TIME =", dateTimeOptions), (0, combi_1.seq)("DATE =", dateTimeOptions), (0, combi_1.seq)("CASE =", caseOptions), (0, combi_1.seq)("EXPONENT", _1.Source), (0, combi_1.seq)("ZERO =", zeroXSDOptions), xsd, (0, combi_1.seq)("STYLE =", styleOptions), (0, combi_1.seq)("CURRENCY =", _1.Source), (0, combi_1.per)(sign, number, decimals, width, pad, alpha, align, country), (0, combi_1.per)(timezone, timestamp));
+        const formatting = (0, combi_1.altPrio)((0, combi_1.seq)("TIME =", dateTimeOptions), (0, combi_1.seq)("DATE =", dateTimeOptions), (0, combi_1.seq)("CASE =", caseOptions), (0, combi_1.seq)("ZERO =", zeroXSDOptions), xsd, (0, combi_1.seq)("STYLE =", styleOptions), (0, combi_1.seq)("CURRENCY =", _1.Source), (0, combi_1.per)(sign, number, decimals, width, pad, alpha, align, country, (0, combi_1.seq)("EXPONENT =", _1.Source)), (0, combi_1.per)(timezone, timestamp));
         return formatting;
     }
 }
@@ -20506,6 +20514,21 @@ exports.sta = sta;
 exports.sub = sub;
 const nodes_1 = __webpack_require__(/*! ../../nodes */ "../core/build/src/abap/nodes/index.js");
 const _statement_1 = __webpack_require__(/*! ../../2_statements/statements/_statement */ "../core/build/src/abap/2_statements/statements/_statement.js");
+function toMatch(statements, m) {
+    return {
+        matched: statements.slice(0, m.next),
+        unmatched: statements.slice(m.next),
+        error: m.error,
+        errorDescription: m.errorDescription,
+        errorMatched: m.errorMatched,
+    };
+}
+function success(next) {
+    return { next, error: false, errorDescription: "", errorMatched: 0 };
+}
+function failure(next, errorDescription, errorMatched) {
+    return { next, error: true, errorDescription, errorMatched };
+}
 class Sequence {
     constructor(list) {
         if (list.length < 2) {
@@ -20524,36 +20547,18 @@ class Sequence {
         return this.list[0].first();
     }
     run(statements, parent) {
-        let inn = statements;
-        let out = [];
+        return toMatch(statements, this.runAt(statements, 0, parent));
+    }
+    runAt(statements, index, parent) {
+        let next = index;
         for (const i of this.list) {
-            const match = i.run(inn, parent);
+            const match = i.runAt(statements, next, parent);
             if (match.error) {
-                return {
-                    matched: [],
-                    unmatched: statements,
-                    error: true,
-                    errorDescription: match.errorDescription,
-                    errorMatched: out.length,
-                };
+                return failure(index, match.errorDescription, next - index);
             }
-            if (match.matched.length < 100) {
-                out.push(...match.matched);
-            }
-            else {
-                // avoid using the spread operator, it might trigger "Maximum call stack size exceeded"
-                // when the number of matched elements is very large
-                out = out.concat(match.matched);
-            }
-            inn = match.unmatched;
+            next = match.next;
         }
-        return {
-            matched: out,
-            unmatched: inn,
-            error: false,
-            errorDescription: "",
-            errorMatched: 0,
-        };
+        return success(next);
     }
 }
 // Note that the Alternative does not nessesarily return the first in the alternative
@@ -20592,21 +20597,18 @@ class Alternative {
         return this.list.reduce((a, c) => { return a.concat(c.getUsing()); }, []);
     }
     run(statements, parent) {
+        return toMatch(statements, this.runAt(statements, 0, parent));
+    }
+    runAt(statements, index, parent) {
         this.setupMap();
         let count = 0;
         let countError = "";
-        if (statements.length === 0) {
-            return {
-                matched: [],
-                unmatched: statements,
-                error: true,
-                errorDescription: countError,
-                errorMatched: count,
-            };
+        if (index >= statements.length) {
+            return failure(index, countError, count);
         }
-        const token = statements[0].getFirstToken().getStr().toUpperCase();
+        const token = statements[index].getFirstToken().getStr().toUpperCase();
         for (const i of this.map[token] || []) {
-            const match = i.run(statements, parent);
+            const match = i.runAt(statements, index, parent);
             if (match.error === false) {
                 return match;
             }
@@ -20616,7 +20618,7 @@ class Alternative {
             }
         }
         for (const i of this.map[""] || []) {
-            const match = i.run(statements, parent);
+            const match = i.runAt(statements, index, parent);
             if (match.error === false) {
                 return match;
             }
@@ -20626,22 +20628,10 @@ class Alternative {
             }
         }
         if (count === 0) {
-            return {
-                matched: [],
-                unmatched: statements,
-                error: true,
-                errorDescription: "Unexpected code structure",
-                errorMatched: count,
-            };
+            return failure(index, "Unexpected code structure", count);
         }
         else {
-            return {
-                matched: [],
-                unmatched: statements,
-                error: true,
-                errorDescription: countError,
-                errorMatched: count,
-            };
+            return failure(index, countError, count);
         }
     }
 }
@@ -20656,7 +20646,10 @@ class Optional {
         return this.obj.getUsing();
     }
     run(statements, parent) {
-        const ret = this.obj.run(statements, parent);
+        return toMatch(statements, this.runAt(statements, 0, parent));
+    }
+    runAt(statements, index, parent) {
+        const ret = this.obj.runAt(statements, index, parent);
         ret.error = false;
         return ret;
     }
@@ -20675,48 +20668,24 @@ class Star {
         return this.obj.getUsing();
     }
     run(statements, parent) {
-        let inn = statements;
-        let out = [];
+        return toMatch(statements, this.runAt(statements, 0, parent));
+    }
+    runAt(statements, index, parent) {
+        let next = index;
         while (true) {
-            if (inn.length === 0) {
-                return {
-                    matched: out,
-                    unmatched: inn,
-                    error: false,
-                    errorDescription: "",
-                    errorMatched: 0,
-                };
+            if (next >= statements.length) {
+                return success(next);
             }
-            const match = this.obj.run(inn, parent);
+            const match = this.obj.runAt(statements, next, parent);
             if (match.error === true) {
                 if (match.errorMatched > 0) {
-                    return {
-                        matched: out,
-                        unmatched: inn,
-                        error: true,
-                        errorDescription: match.errorDescription,
-                        errorMatched: match.errorMatched,
-                    };
+                    return failure(next, match.errorDescription, match.errorMatched);
                 }
                 else {
-                    return {
-                        matched: out,
-                        unmatched: inn,
-                        error: false,
-                        errorDescription: "",
-                        errorMatched: 0,
-                    };
+                    return success(next);
                 }
             }
-            if (match.matched.length < 100) {
-                out.push(...match.matched);
-            }
-            else {
-                // avoid using the spread operator, it might trigger "Maximum call stack size exceeded"
-                // when the number of matched elements is very large
-                out = out.concat(match.matched);
-            }
-            inn = match.unmatched;
+            next = match.next;
         }
     }
     first() {
@@ -20745,10 +20714,13 @@ class SubStructure {
         }
     }
     run(statements, parent) {
+        return toMatch(statements, this.runAt(statements, 0, parent));
+    }
+    runAt(statements, index, parent) {
         const nparent = new nodes_1.StructureNode(this.s);
         this.setupMatcher();
-        const ret = this.matcher.run(statements, nparent);
-        if (ret.matched.length === 0) {
+        const ret = this.matcher.runAt(statements, index, nparent);
+        if (ret.next === index) {
             ret.error = true;
         }
         else {
@@ -20778,33 +20750,18 @@ class SubStatement {
         return this.obj.name;
     }
     run(statements, parent) {
-        if (statements.length === 0) {
-            return {
-                matched: [],
-                unmatched: [],
-                error: true,
-                errorDescription: "Expected " + this.className().toUpperCase(),
-                errorMatched: 0,
-            };
+        return toMatch(statements, this.runAt(statements, 0, parent));
+    }
+    runAt(statements, index, parent) {
+        if (index >= statements.length) {
+            return failure(statements.length, "Expected " + this.className().toUpperCase(), 0);
         }
-        else if (statements[0].get() instanceof this.obj) {
-            parent.addChild(statements[0]);
-            return {
-                matched: [statements[0]],
-                unmatched: statements.splice(1),
-                error: false,
-                errorDescription: "",
-                errorMatched: 0,
-            };
+        else if (statements[index].get() instanceof this.obj) {
+            parent.addChild(statements[index]);
+            return success(index + 1);
         }
         else {
-            return {
-                matched: [],
-                unmatched: statements,
-                error: true,
-                errorDescription: "Expected " + this.className().toUpperCase(),
-                errorMatched: 0,
-            };
+            return failure(index, "Expected " + this.className().toUpperCase(), 0);
         }
     }
 }
@@ -28550,7 +28507,7 @@ class BasicTypes {
         if (children.length === 0) {
             return new Types.UnknownType("Type error, could not resolve \"" + fullName + "\", resolveLikeName1");
         }
-        let type = undefined;
+        let type;
         if (children[1] && (children[1].getFirstToken().getStr() === "=>" || children[1].getFirstToken().getStr() === "->")) {
             type = field_chain_1.FieldChain.runSyntax(chain, this.input, _reference_1.ReferenceType.TypeReference);
         }
@@ -28813,7 +28770,7 @@ class BasicTypes {
             primaryKey: primaryKey,
             secondary: secondaryKeys,
         };
-        let found = undefined;
+        let found;
         if (text.startsWith("TYPE TABLE OF REF TO ")
             || text.startsWith("TYPE STANDARD TABLE OF REF TO ")
             || text.startsWith("TYPE SORTED TABLE OF REF TO ")
@@ -28951,7 +28908,7 @@ class BasicTypes {
                 name = typeName === null || typeName === void 0 ? void 0 : typeName.concatTokens();
             }
             const type = this.resolveLikeName(e, false);
-            let row = undefined;
+            let row;
             if (type === undefined) {
                 return new Types.UnknownType("Type error, could not resolve \"" + name + "\", parseType");
             }
@@ -29146,7 +29103,7 @@ class BasicTypes {
             rest = split[1];
         }
         const subs = rest.split("-");
-        let foundType = undefined;
+        let foundType;
         if (className && chainText.includes("=>")) {
             const split = chainText.split("=>");
             const className = split[0];
@@ -29569,6 +29526,143 @@ function checkDatabaseFields(fields, dbSources, input, token) {
 
 /***/ },
 
+/***/ "../core/build/src/abap/5_syntax/expressions/_check_database_work_area.js"
+/*!********************************************************************************!*\
+  !*** ../core/build/src/abap/5_syntax/expressions/_check_database_work_area.js ***!
+  \********************************************************************************/
+(__unused_webpack_module, exports, __webpack_require__) {
+
+"use strict";
+
+var __createBinding = (this && this.__createBinding) || (Object.create ? (function(o, m, k, k2) {
+    if (k2 === undefined) k2 = k;
+    var desc = Object.getOwnPropertyDescriptor(m, k);
+    if (!desc || ("get" in desc ? !m.__esModule : desc.writable || desc.configurable)) {
+      desc = { enumerable: true, get: function() { return m[k]; } };
+    }
+    Object.defineProperty(o, k2, desc);
+}) : (function(o, m, k, k2) {
+    if (k2 === undefined) k2 = k;
+    o[k2] = m[k];
+}));
+var __setModuleDefault = (this && this.__setModuleDefault) || (Object.create ? (function(o, v) {
+    Object.defineProperty(o, "default", { enumerable: true, value: v });
+}) : function(o, v) {
+    o["default"] = v;
+});
+var __importStar = (this && this.__importStar) || (function () {
+    var ownKeys = function(o) {
+        ownKeys = Object.getOwnPropertyNames || function (o) {
+            var ar = [];
+            for (var k in o) if (Object.prototype.hasOwnProperty.call(o, k)) ar[ar.length] = k;
+            return ar;
+        };
+        return ownKeys(o);
+    };
+    return function (mod) {
+        if (mod && mod.__esModule) return mod;
+        var result = {};
+        if (mod != null) for (var k = ownKeys(mod), i = 0; i < k.length; i++) if (k[i] !== "default") __createBinding(result, mod, k[i]);
+        __setModuleDefault(result, mod);
+        return result;
+    };
+})();
+Object.defineProperty(exports, "__esModule", ({ value: true }));
+exports.checkDatabaseWorkArea = checkDatabaseWorkArea;
+const nodes_1 = __webpack_require__(/*! ../../nodes */ "../core/build/src/abap/nodes/index.js");
+const Expressions = __importStar(__webpack_require__(/*! ../../2_statements/expressions */ "../core/build/src/abap/2_statements/expressions/index.js"));
+const Basic = __importStar(__webpack_require__(/*! ../../types/basic */ "../core/build/src/abap/types/basic/index.js"));
+const objects_1 = __webpack_require__(/*! ../../../objects */ "../core/build/src/objects/index.js");
+const _syntax_input_1 = __webpack_require__(/*! ../_syntax_input */ "../core/build/src/abap/5_syntax/_syntax_input.js");
+// byte length of a flat type in a unicode system, as a lower and an upper bound:
+// IntegerType also stands for INT1 and INT2, and padding is left out of the lower bound
+function bounds(type) {
+    if (type instanceof Basic.CharacterType || type instanceof Basic.NumericType) {
+        return { min: type.getLength() * 2, max: type.getLength() * 2, align: 2 };
+    }
+    else if (type instanceof Basic.DateType) {
+        return { min: 16, max: 16, align: 2 };
+    }
+    else if (type instanceof Basic.TimeType) {
+        return { min: 12, max: 12, align: 2 };
+    }
+    else if (type instanceof Basic.IntegerType) {
+        return { min: 1, max: 4, align: 4 };
+    }
+    else if (type instanceof Basic.Integer8Type || type instanceof Basic.FloatType
+        || type instanceof Basic.DecFloat16Type || type instanceof Basic.UTCLongType) {
+        return { min: 8, max: 8, align: 8 };
+    }
+    else if (type instanceof Basic.DecFloat34Type) {
+        return { min: 16, max: 16, align: 16 };
+    }
+    else if (type instanceof Basic.PackedType || type instanceof Basic.HexType) {
+        return { min: type.getLength(), max: type.getLength(), align: 1 };
+    }
+    else if (type instanceof Basic.StructureType) {
+        let min = 0;
+        let max = 0;
+        let align = 1;
+        for (const component of type.getComponents()) {
+            const sub = bounds(component.type);
+            if (sub === undefined) {
+                return undefined;
+            }
+            align = Math.max(align, sub.align);
+            max = Math.ceil(max / sub.align) * sub.align + sub.max;
+            min = min + sub.min;
+        }
+        max = Math.ceil(max / align) * align;
+        return { min, max, align };
+    }
+    // deep, generic, void or unknown
+    return undefined;
+}
+// INSERT/MODIFY/UPDATE dbtab FROM wa: the work area must be at least as long as the table line,
+// client field included, else the statement is a syntax error on the system
+function checkDatabaseWorkArea(node, dbSource, sourceTypes, input) {
+    var _a;
+    if (!(dbSource instanceof objects_1.Table)) {
+        return;
+    }
+    const children = node.getChildren();
+    let workArea = undefined;
+    for (let i = 0; i < children.length - 1; i++) {
+        const child = children[i];
+        if (child instanceof nodes_1.TokenNode && child.concatTokens().toUpperCase() === "FROM") {
+            const next = children[i + 1];
+            if (next instanceof nodes_1.ExpressionNode && next.get() instanceof Expressions.SQLSource) {
+                workArea = next;
+            }
+            break;
+        }
+    }
+    if (workArea === undefined) {
+        return;
+    }
+    const source = (_a = workArea.findFirstExpression(Expressions.SimpleSource3)) !== null && _a !== void 0 ? _a : workArea.findFirstExpression(Expressions.Source);
+    if (source === undefined) {
+        return;
+    }
+    const waType = sourceTypes.get(source);
+    if (!(waType instanceof Basic.StructureType)) {
+        return;
+    }
+    const tableType = dbSource.parseType(input.scope.getRegistry());
+    const waBounds = bounds(waType);
+    const tableBounds = bounds(tableType);
+    if (waBounds === undefined || tableBounds === undefined) {
+        return;
+    }
+    if (waBounds.max < tableBounds.min) {
+        const message = "The work area \"" + source.concatTokens() + "\" is not long enough for table " + dbSource.getName();
+        input.issues.push((0, _syntax_input_1.syntaxIssue)(input, source.getFirstToken(), message));
+    }
+}
+//# sourceMappingURL=_check_database_work_area.js.map
+
+/***/ },
+
 /***/ "../core/build/src/abap/5_syntax/expressions/_check_offset_length.js"
 /*!***************************************************************************!*\
   !*** ../core/build/src/abap/5_syntax/expressions/_check_offset_length.js ***!
@@ -29890,7 +29984,7 @@ class AttributeName {
             return context;
         }
         const helper = new _object_oriented_1.ObjectOriented(input.scope);
-        let ret = undefined;
+        let ret;
         if (context instanceof object_reference_type_1.ObjectReferenceType) {
             const def = input.scope.findObjectDefinition(context.getIdentifierName());
             if (def === undefined) {
@@ -30121,12 +30215,15 @@ var __importStar = (this && this.__importStar) || (function () {
 Object.defineProperty(exports, "__esModule", ({ value: true }));
 exports.Compare = void 0;
 const Expressions = __importStar(__webpack_require__(/*! ../../2_statements/expressions */ "../core/build/src/abap/2_statements/expressions/index.js"));
+const nodes_1 = __webpack_require__(/*! ../../nodes */ "../core/build/src/abap/nodes/index.js");
 const source_1 = __webpack_require__(/*! ./source */ "../core/build/src/abap/5_syntax/expressions/source.js");
 const method_call_chain_1 = __webpack_require__(/*! ./method_call_chain */ "../core/build/src/abap/5_syntax/expressions/method_call_chain.js");
 const source_field_symbol_1 = __webpack_require__(/*! ./source_field_symbol */ "../core/build/src/abap/5_syntax/expressions/source_field_symbol.js");
 const _syntax_input_1 = __webpack_require__(/*! ../_syntax_input */ "../core/build/src/abap/5_syntax/_syntax_input.js");
 const basic_1 = __webpack_require__(/*! ../../types/basic */ "../core/build/src/abap/types/basic/index.js");
 const _type_utils_1 = __webpack_require__(/*! ../_type_utils */ "../core/build/src/abap/5_syntax/_type_utils.js");
+const _builtin_1 = __webpack_require__(/*! ../_builtin */ "../core/build/src/abap/5_syntax/_builtin.js");
+const _object_oriented_1 = __webpack_require__(/*! ../_object_oriented */ "../core/build/src/abap/5_syntax/_object_oriented.js");
 class Compare {
     static runSyntax(node, input) {
         var _a;
@@ -30141,6 +30238,11 @@ class Compare {
         for (const t of node.findDirectExpressions(Expressions.MethodCallChain)) {
             method_call_chain_1.MethodCallChain.runSyntax(t, input);
         }
+        const initialMessage = this.checkInitialOperand(node, input);
+        if (initialMessage !== undefined) {
+            input.issues.push((0, _syntax_input_1.syntaxIssue)(input, node.getFirstToken(), initialMessage));
+            return;
+        }
         const typeUtils = new _type_utils_1.TypeUtils(input.scope);
         const compareOperator = (_a = node.findDirectExpression(Expressions.CompareOperator)) === null || _a === void 0 ? void 0 : _a.concatTokens().toUpperCase();
         if (compareOperator === "CS"
@@ -30164,6 +30266,46 @@ class Compare {
             input.issues.push((0, _syntax_input_1.syntaxIssue)(input, node.getFirstToken(), message));
         }
     }
+    // a built-in function with a character-like result cannot be the operand of IS [NOT] INITIAL,
+    // "Unexpected operator IS" on a system, while one with a numeric result, lines( ), can
+    static checkInitialOperand(node, input) {
+        var _a, _b, _c, _d;
+        if (input.scope.getOpenABAP()) {
+            return undefined;
+        }
+        const tokens = node.getChildren().filter(c => c instanceof nodes_1.TokenNode).map(c => c.getFirstToken().getStr().toUpperCase());
+        if (tokens.includes("IS") === false || tokens[tokens.length - 1] !== "INITIAL") {
+            return undefined;
+        }
+        const chain = (_a = node.findDirectExpression(Expressions.Source)) === null || _a === void 0 ? void 0 : _a.getChildren();
+        // xsdbool( ) and boolc( ) are their own form in the Source grammar, a condition in parentheses
+        const first = (_b = chain === null || chain === void 0 ? void 0 : chain[0]) === null || _b === void 0 ? void 0 : _b.getFirstToken().getStr().toUpperCase();
+        if ((chain === null || chain === void 0 ? void 0 : chain.length) === 4 && chain[0] instanceof nodes_1.TokenNode && (first === "XSDBOOL" || first === "BOOLC")) {
+            return `Built-in function "${chain[0].getFirstToken().getStr()}" cannot be the operand of IS INITIAL`;
+        }
+        if ((chain === null || chain === void 0 ? void 0 : chain.length) !== 1 || !(chain[0] instanceof nodes_1.ExpressionNode) || !(chain[0].get() instanceof Expressions.MethodCallChain)) {
+            return undefined;
+        }
+        const calls = chain[0].getChildren();
+        if (calls.length !== 1 || !(calls[0] instanceof nodes_1.ExpressionNode) || !(calls[0].get() instanceof Expressions.MethodCall)) {
+            return undefined;
+        }
+        const name = (_c = calls[0].findDirectExpression(Expressions.MethodName)) === null || _c === void 0 ? void 0 : _c.concatTokens();
+        const builtin = _builtin_1.BuiltIn.searchBuiltin(name);
+        if (name === undefined || builtin === undefined) {
+            return undefined;
+        }
+        // a method of the same name hides the built-in function
+        const enclosing = input.scope.findClassDefinition(input.scope.getEnclosingClassName());
+        if (enclosing !== undefined && new _object_oriented_1.ObjectOriented(input.scope).searchMethodName(enclosing, name).method !== undefined) {
+            return undefined;
+        }
+        const returning = (_d = builtin.getParameters().getReturning()) === null || _d === void 0 ? void 0 : _d.getType();
+        if (returning instanceof basic_1.StringType || returning instanceof basic_1.CharacterType) {
+            return `Built-in function "${name}" cannot be the operand of IS INITIAL`;
+        }
+        return undefined;
+    }
 }
 exports.Compare = Compare;
 //# sourceMappingURL=compare.js.map
@@ -30867,7 +31009,7 @@ const let_1 = __webpack_require__(/*! ./let */ "../core/build/src/abap/5_syntax/
 const _syntax_input_1 = __webpack_require__(/*! ../_syntax_input */ "../core/build/src/abap/5_syntax/_syntax_input.js");
 const basic_1 = __webpack_require__(/*! ../../types/basic */ "../core/build/src/abap/types/basic/index.js");
 class ConvBody {
-    static runSyntax(node, input) {
+    static runSyntax(node, input, targetType) {
         let scoped = false;
         const l = node.findDirectExpression(Expressions.Let);
         if (l) {
@@ -30879,7 +31021,7 @@ class ConvBody {
             input.issues.push((0, _syntax_input_1.syntaxIssue)(input, node.getFirstToken(), message));
             return basic_1.VoidType.get(_syntax_input_1.CheckSyntaxKey);
         }
-        const sourceType = source_1.Source.runSyntax(s, input);
+        const sourceType = source_1.Source.runSyntax(s, input, targetType);
         if (scoped === true) {
             input.scope.pop(node.getLastToken().getEnd());
         }
@@ -31412,7 +31554,7 @@ class FieldChain {
                 return found.getType();
             }
         }
-        let context = undefined;
+        let context;
         const children = node.getChildren();
         context = this.findTop(children[0], input, refType);
         let prev = children[0].concatTokens();
@@ -31905,7 +32047,7 @@ class FormParam {
         if (node.findDirectTokenByText("STRUCTURE") && nameToken) {
             // STRUCTURES typing
             const typeName = (_a = node.findDirectExpression(expressions_1.SimpleFieldChain)) === null || _a === void 0 ? void 0 : _a.getFirstToken().getStr();
-            let type = undefined;
+            let type;
             if (typeName) {
                 type = (_b = input.scope.findType(typeName)) === null || _b === void 0 ? void 0 : _b.getType();
                 if (type === undefined) {
@@ -33478,7 +33620,7 @@ class MethodParameters {
     }
     checkImporting(node, input, method) {
         for (const item of this.parameterListT(node, input)) {
-            let parameterType = undefined;
+            let parameterType;
             if (method instanceof basic_1.VoidType) {
                 parameterType = method;
             }
@@ -33515,7 +33657,7 @@ class MethodParameters {
                 input.issues.push((0, _syntax_input_1.syntaxIssue)(input, node.getFirstToken(), message));
                 return;
             }
-            let parameterType = undefined;
+            let parameterType;
             if (method instanceof basic_1.VoidType) {
                 parameterType = method;
             }
@@ -34832,9 +34974,8 @@ class Select {
     }
     static findFields(node, input) {
         var _a, _b;
-        let expr = undefined;
+        const expr = node.findFirstExpression(Expressions.SQLFieldList);
         const ret = [];
-        expr = node.findFirstExpression(Expressions.SQLFieldList);
         if (((_a = expr === null || expr === void 0 ? void 0 : expr.getFirstChild()) === null || _a === void 0 ? void 0 : _a.get()) instanceof Expressions.Dynamic) {
             dynamic_1.Dynamic.runSyntax(expr.getFirstChild(), input);
         }
@@ -34928,6 +35069,7 @@ const _typed_identifier_1 = __webpack_require__(/*! ../../types/_typed_identifie
 const _type_utils_1 = __webpack_require__(/*! ../_type_utils */ "../core/build/src/abap/5_syntax/_type_utils.js");
 const _syntax_input_1 = __webpack_require__(/*! ../_syntax_input */ "../core/build/src/abap/5_syntax/_syntax_input.js");
 const assert_error_1 = __webpack_require__(/*! ../assert_error */ "../core/build/src/abap/5_syntax/assert_error.js");
+const version_1 = __webpack_require__(/*! ../../../version */ "../core/build/src/version.js");
 /*
 * Type interference, valid scenarios:
 * typed = VALUE #( ... ).         right hand side must follow left hand type
@@ -34940,7 +35082,7 @@ const assert_error_1 = __webpack_require__(/*! ../assert_error */ "../core/build
 // TODO: refactor all these method parameters to objects, this is getting messy
 class Source {
     static runSyntax(node, input, targetType, writeReference = false, allowGenericDeference = false, readReference = true) {
-        var _a;
+        var _a, _b, _c, _d;
         if (node === undefined) {
             return undefined;
         }
@@ -35018,7 +35160,7 @@ class Source {
                 case "CONV":
                     {
                         const foundType = this.determineType(node, input, targetType);
-                        const bodyType = conv_body_1.ConvBody.runSyntax(node.findDirectExpression(Expressions.ConvBody), input);
+                        const bodyType = conv_body_1.ConvBody.runSyntax(node.findDirectExpression(Expressions.ConvBody), input, foundType);
                         const inferred = (_a = node.findDirectExpression(Expressions.TypeNameOrInfer)) === null || _a === void 0 ? void 0 : _a.concatTokens();
                         if (new _type_utils_1.TypeUtils(input.scope).isConvable(foundType, bodyType) === false) {
                             const message = `CONV: Types not compatible, ${foundType === null || foundType === void 0 ? void 0 : foundType.constructor.name}, ${bodyType === null || bodyType === void 0 ? void 0 : bodyType.constructor.name}`;
@@ -35035,7 +35177,13 @@ class Source {
                             this.traverseRemainingChildren(children, input);
                             return bodyType;
                         }
-                        this.addIfInferred(node, input, foundType);
+                        const bodySource = (_b = node.findDirectExpression(Expressions.ConvBody)) === null || _b === void 0 ? void 0 : _b.findDirectExpression(Expressions.Source);
+                        const nestedType = (_c = bodySource === null || bodySource === void 0 ? void 0 : bodySource.findDirectExpression(Expressions.TypeNameOrInfer)) === null || _c === void 0 ? void 0 : _c.concatTokens();
+                        // CONV # passes its target type into a nested inferred constructor, so
+                        // showing both inferred types would produce duplicate inlay hints.
+                        if (!(inferred === "#" && nestedType === "#")) {
+                            this.addIfInferred(node, input, foundType);
+                        }
                         this.traverseRemainingChildren(children, input);
                         return foundType;
                     }
@@ -35092,6 +35240,15 @@ class Source {
                 case "CORRESPONDING":
                     {
                         const foundType = this.determineType(node, input, targetType);
+                        if ((foundType === null || foundType === void 0 ? void 0 : foundType.isGeneric()) === true
+                            && ((_d = node.findDirectExpression(Expressions.TypeNameOrInfer)) === null || _d === void 0 ? void 0 : _d.concatTokens()) === "#"
+                            && !(0, version_1.releaseAtLeast)(input.scope.getRelease(), version_1.Release.v756)
+                            && input.scope.getLanguageVersion() !== version_1.LanguageVersion.Cloud
+                            && !input.scope.getOpenABAP()) {
+                            const message = "CORRESPONDING #, the type of the target is generic";
+                            input.issues.push((0, _syntax_input_1.syntaxIssue)(input, node.getFirstToken(), message));
+                            return basic_1.VoidType.get(_syntax_input_1.CheckSyntaxKey);
+                        }
                         corresponding_body_1.CorrespondingBody.runSyntax(node.findDirectExpression(Expressions.CorrespondingBody), input, foundType);
                         this.addIfInferred(node, input, foundType);
                         return foundType;
@@ -37737,6 +37894,7 @@ const basic_1 = __webpack_require__(/*! ../../types/basic */ "../core/build/src/
 const dynamic_1 = __webpack_require__(/*! ../expressions/dynamic */ "../core/build/src/abap/5_syntax/expressions/dynamic.js");
 const _type_utils_1 = __webpack_require__(/*! ../_type_utils */ "../core/build/src/abap/5_syntax/_type_utils.js");
 const _syntax_input_1 = __webpack_require__(/*! ../_syntax_input */ "../core/build/src/abap/5_syntax/_syntax_input.js");
+const version_1 = __webpack_require__(/*! ../../../version */ "../core/build/src/version.js");
 class Assign {
     runSyntax(node, input) {
         var _a;
@@ -37744,7 +37902,7 @@ class Assign {
         const isComponent = ((_a = assignSource === null || assignSource === void 0 ? void 0 : assignSource.getFirstChild()) === null || _a === void 0 ? void 0 : _a.concatTokens().toUpperCase()) === "COMPONENT";
         const sources = (assignSource === null || assignSource === void 0 ? void 0 : assignSource.findDirectExpressionsMulti([Expressions.Source, Expressions.SimpleSource3])) || [];
         const theSource = sources[sources.length - 1];
-        let sourceType = undefined;
+        let sourceType;
         const firstAssign = assignSource === null || assignSource === void 0 ? void 0 : assignSource.getChildren()[0];
         const secondAssign = assignSource === null || assignSource === void 0 ? void 0 : assignSource.getChildren()[1];
         const thirdAssign = assignSource === null || assignSource === void 0 ? void 0 : assignSource.getChildren()[2];
@@ -37760,6 +37918,19 @@ class Assign {
         }
         else {
             sourceType = source_1.Source.runSyntax(theSource, input, undefined, false, isComponent === false);
+            if ((secondAssign === null || secondAssign === void 0 ? void 0 : secondAssign.concatTokens()) === "->"
+                && (thirdAssign === null || thirdAssign === void 0 ? void 0 : thirdAssign.get()) instanceof Expressions.Dynamic
+                && (sourceType === null || sourceType === void 0 ? void 0 : sourceType.isGeneric()) === true
+                && !(sourceType instanceof basic_1.DataReference)
+                && !(sourceType instanceof basic_1.ObjectReferenceType)
+                && !(sourceType instanceof basic_1.GenericObjectReferenceType)
+                && !(0, version_1.releaseAtLeast)(input.scope.getRelease(), version_1.Release.v756)
+                && input.scope.getLanguageVersion() !== version_1.LanguageVersion.Cloud
+                && !input.scope.getOpenABAP()) {
+                const message = `"${theSource.concatTokens()}" is not a reference variable`;
+                input.issues.push((0, _syntax_input_1.syntaxIssue)(input, node.getFirstToken(), message));
+                return;
+            }
         }
         if ((assignSource === null || assignSource === void 0 ? void 0 : assignSource.getChildren().length) === 5
             && isComponent) {
@@ -42486,20 +42657,23 @@ const Expressions = __importStar(__webpack_require__(/*! ../../2_statements/expr
 const source_1 = __webpack_require__(/*! ../expressions/source */ "../core/build/src/abap/5_syntax/expressions/source.js");
 const dynamic_1 = __webpack_require__(/*! ../expressions/dynamic */ "../core/build/src/abap/5_syntax/expressions/dynamic.js");
 const database_table_1 = __webpack_require__(/*! ../expressions/database_table */ "../core/build/src/abap/5_syntax/expressions/database_table.js");
+const _check_database_work_area_1 = __webpack_require__(/*! ../expressions/_check_database_work_area */ "../core/build/src/abap/5_syntax/expressions/_check_database_work_area.js");
 class InsertDatabase {
     runSyntax(node, input) {
+        const sourceTypes = new Map();
         for (const s of node.findAllExpressions(Expressions.Source)) {
-            source_1.Source.runSyntax(s, input);
+            sourceTypes.set(s, source_1.Source.runSyntax(s, input));
         }
         for (const s of node.findAllExpressions(Expressions.SimpleSource3)) {
-            source_1.Source.runSyntax(s, input);
+            sourceTypes.set(s, source_1.Source.runSyntax(s, input));
         }
         for (const d of node.findAllExpressions(Expressions.Dynamic)) {
             dynamic_1.Dynamic.runSyntax(d, input);
         }
         const dbtab = node.findFirstExpression(Expressions.DatabaseTable);
         if (dbtab !== undefined) {
-            database_table_1.DatabaseTable.runSyntax(dbtab, input);
+            const dbSource = database_table_1.DatabaseTable.runSyntax(dbtab, input);
+            (0, _check_database_work_area_1.checkDatabaseWorkArea)(node, dbSource, sourceTypes, input);
         }
     }
 }
@@ -43085,7 +43259,7 @@ class Loop {
             && topType instanceof basic_1.TableType) {
             // https://github.com/abap2xlsx/abap2xlsx/issues/1341
             const keyName = node.findExpressionAfterToken("KEY");
-            let key = undefined;
+            let key;
             const name = (keyName === null || keyName === void 0 ? void 0 : keyName.get()) instanceof Expressions.SimpleName ? keyName.getFirstToken().getStr().toUpperCase() : undefined;
             // it might be dynamic, in that case we cannot check anything; and
             // primary_key is the predefined name of the primary key every table has
@@ -43442,11 +43616,13 @@ const dynamic_1 = __webpack_require__(/*! ../expressions/dynamic */ "../core/bui
 const database_table_1 = __webpack_require__(/*! ../expressions/database_table */ "../core/build/src/abap/5_syntax/expressions/database_table.js");
 const source_1 = __webpack_require__(/*! ../expressions/source */ "../core/build/src/abap/5_syntax/expressions/source.js");
 const _reference_1 = __webpack_require__(/*! ../_reference */ "../core/build/src/abap/5_syntax/_reference.js");
+const _check_database_work_area_1 = __webpack_require__(/*! ../expressions/_check_database_work_area */ "../core/build/src/abap/5_syntax/expressions/_check_database_work_area.js");
 class ModifyDatabase {
     runSyntax(node, input) {
         for (const d of node.findAllExpressions(Expressions.Dynamic)) {
             dynamic_1.Dynamic.runSyntax(d, input);
         }
+        let dbSource = undefined;
         const dbtab = node.findFirstExpression(Expressions.DatabaseTable);
         if (dbtab !== undefined) {
             if (node.getChildren().length === 5) {
@@ -43455,19 +43631,21 @@ class ModifyDatabase {
                     input.scope.addReference(dbtab.getFirstToken(), found, _reference_1.ReferenceType.DataWriteReference, input.filename);
                 }
                 else {
-                    database_table_1.DatabaseTable.runSyntax(dbtab, input);
+                    dbSource = database_table_1.DatabaseTable.runSyntax(dbtab, input);
                 }
             }
             else {
-                database_table_1.DatabaseTable.runSyntax(dbtab, input);
+                dbSource = database_table_1.DatabaseTable.runSyntax(dbtab, input);
             }
         }
+        const sourceTypes = new Map();
         for (const s of node.findAllExpressions(Expressions.Source)) {
-            source_1.Source.runSyntax(s, input);
+            sourceTypes.set(s, source_1.Source.runSyntax(s, input));
         }
         for (const s of node.findAllExpressions(Expressions.SimpleSource3)) {
-            source_1.Source.runSyntax(s, input);
+            sourceTypes.set(s, source_1.Source.runSyntax(s, input));
         }
+        (0, _check_database_work_area_1.checkDatabaseWorkArea)(node, dbSource, sourceTypes, input);
     }
 }
 exports.ModifyDatabase = ModifyDatabase;
@@ -47367,11 +47545,13 @@ const identifier_1 = __webpack_require__(/*! ../../1_lexer/tokens/identifier */
 const database_table_1 = __webpack_require__(/*! ../expressions/database_table */ "../core/build/src/abap/5_syntax/expressions/database_table.js");
 const dynamic_1 = __webpack_require__(/*! ../expressions/dynamic */ "../core/build/src/abap/5_syntax/expressions/dynamic.js");
 const _check_database_fields_1 = __webpack_require__(/*! ../expressions/_check_database_fields */ "../core/build/src/abap/5_syntax/expressions/_check_database_fields.js");
+const _check_database_work_area_1 = __webpack_require__(/*! ../expressions/_check_database_work_area */ "../core/build/src/abap/5_syntax/expressions/_check_database_work_area.js");
 class UpdateDatabase {
     runSyntax(node, input) {
+        let dbSource = undefined;
         const dbtab = node.findFirstExpression(Expressions.DatabaseTable);
         if (dbtab !== undefined) {
-            const dbSource = database_table_1.DatabaseTable.runSyntax(dbtab, input);
+            dbSource = database_table_1.DatabaseTable.runSyntax(dbtab, input);
             const fields = node.findAllExpressions(Expressions.SQLFieldAndValue)
                 .flatMap(fieldAndValue => fieldAndValue.findDirectExpressions(Expressions.SQLFieldName))
                 .map(field => field.concatTokens().toUpperCase());
@@ -47394,15 +47574,17 @@ class UpdateDatabase {
                 }
             }
         }
+        const sourceTypes = new Map();
         for (const s of node.findAllExpressions(Expressions.Source)) {
-            source_1.Source.runSyntax(s, input);
+            sourceTypes.set(s, source_1.Source.runSyntax(s, input));
         }
         for (const s of node.findAllExpressions(Expressions.SimpleSource3)) {
-            source_1.Source.runSyntax(s, input);
+            sourceTypes.set(s, source_1.Source.runSyntax(s, input));
         }
         for (const d of node.findAllExpressions(Expressions.Dynamic)) {
             dynamic_1.Dynamic.runSyntax(d, input);
         }
+        (0, _check_database_work_area_1.checkDatabaseWorkArea)(node, dbSource, sourceTypes, input);
         if (input.scope.getType() === _scope_type_1.ScopeType.OpenSQL) {
             input.scope.pop(node.getLastToken().getEnd());
         }
@@ -47591,7 +47773,7 @@ class WhenType {
         if (nameToken === undefined) {
             return undefined;
         }
-        let type = undefined;
+        let type;
         const className = nameToken.getStr().toUpperCase();
         const found = input.scope.existsObject(className);
         if (found === null || found === void 0 ? void 0 : found.id) {
@@ -52715,7 +52897,7 @@ class Attributes {
         return false;
     }
     parseAttribute(node, visibility, input) {
-        let found = undefined;
+        let found;
         const s = node.get();
         if (s instanceof Statements.Data) {
             found = new data_1.Data().runSyntax(node, input);
@@ -54891,7 +55073,7 @@ class CDSLexer {
                     break;
                 case "@":
                     // @ starts a new annotation; flush current token and start building with @
-                    build = result.add(build, row, col, mode);
+                    result.add(build, row, col, mode);
                     build = "@";
                     break;
                 default:
@@ -59765,7 +59947,7 @@ class Help {
         return content;
     }
     static dumpABAP(file, reg, textDocument, position) {
-        let content = "";
+        let content;
         content = `
     <a href="#_tokens" rel="no-refresh">Tokens</a> |
     <a href="#_statements" rel="no-refresh">Statements</a> |
@@ -59798,7 +59980,7 @@ class Help {
         return `<h3 id="_info">Info Dump</h3><pre>` + text + "</pre>";
     }
     static cursorInformation(reg, textDocument, position, file) {
-        let ret = "";
+        let ret;
         const found = _lsp_utils_1.LSPUtils.findCursor(reg, { textDocument, position });
         if (found !== undefined) {
             ret = "Statement: " + this.linkToStatement(found.snode.get()) + "<br>\n" +
@@ -63556,7 +63738,7 @@ class DataElement extends _abstract_object_1.AbstractObject {
     }
     parseType(reg) {
         const references = [];
-        let lookup = undefined;
+        let lookup;
         this.parse();
         if (this.parsedXML === undefined) {
             lookup = { type: new Types.UnknownType("Data Element " + this.getName() + ", parser error") };
@@ -69007,6 +69189,7 @@ class Table extends _abstract_object_1.AbstractObject {
                 PRECFIELD: field.PRECFIELD,
                 LENG: field.LENG,
                 INTLEN: field.INTLEN,
+                INTTYPE: field.INTTYPE,
                 DATATYPE: field.DATATYPE,
                 DECIMALS: field.DECIMALS,
                 KEYFLAG: field.KEYFLAG,
@@ -70525,7 +70708,7 @@ class Registry {
     }
     static abaplintVersion() {
         // magic, see build script "version.js"
-        return "2.120.70";
+        return "2.120.72";
     }
     getDDICReferences() {
         return this.ddicReferences;
@@ -71873,7 +72056,7 @@ class AlignPseudoComments extends _abap_rule_1.ABAPRule {
             }
             const col = firstCommentToken.getStart().getCol();
             if (previousEnd.getCol() < expectedColumn && col !== expectedColumn) {
-                let fix = undefined;
+                let fix;
                 if (col < expectedColumn) {
                     fix = edit_helper_1.EditHelper.insertAt(file, firstCommentToken.getStart(), " ".repeat(expectedColumn - col));
                 }
@@ -72040,7 +72223,7 @@ ENDINTERFACE.`,
             if (f.after.getCol() === column) {
                 continue;
             }
-            let fix = undefined;
+            let fix;
             if (f.after.getCol() < column) {
                 fix = edit_helper_1.EditHelper.insertAt(file, f.after, " ".repeat(column - f.after.getCol()));
             }
@@ -73214,7 +73397,7 @@ exports.CDSCheckSyntaxConf = CDSCheckSyntaxConf;
 const KNOWN_QUAN_DATA_ELEMENTS = ["MENGE_D"];
 const KNOWN_CURR_DATA_ELEMENTS = ["BWERT", "DZWERT"];
 // Reserved names, cannot be used as element names, see DDIC table TRESE
-const RESERVED_ELEMENT_NAMES = ["BEGIN", "NUMBER", "POSITION"];
+const RESERVED_ELEMENT_NAMES = ["BEGIN", "NUMBER", "POSITION", "ZONE", "HANDLER", "SECTION"];
 const MAX_LABEL_LENGTH = 40;
 // Annotations which are not supported in CDS view entities
 const VIEW_ENTITY_FORBIDDEN_ANNOTATIONS = ["Semantics.unitOfMeasure", "Semantics.currencyCode"];
@@ -74431,7 +74614,7 @@ ENDCASE.`,
         if (tuples.length === 1) {
             return false;
         }
-        let chain = "";
+        let chain;
         if (tuples[0].left === tuples[1].left) {
             chain = tuples[0].left;
         }
@@ -74720,7 +74903,7 @@ class CheckDDIC {
         this.conf = conf;
     }
     run(obj) {
-        let found = undefined;
+        let found;
         if (obj instanceof Objects.DataElement
             || obj instanceof Objects.Domain
             || obj instanceof Objects.Table
@@ -75444,7 +75627,6 @@ ENDCLASS.`,
         this.conf = conf;
     }
     runParsed(file, obj) {
-        let issues = [];
         if (this.conf.patternKind === undefined) {
             this.conf.patternKind = "required";
         }
@@ -75465,8 +75647,7 @@ ENDCLASS.`,
                 attributes = attributes.concat(intfDef.attributes);
             }
         }
-        issues = this.checkAttributes(attributes);
-        return issues;
+        return this.checkAttributes(attributes);
     }
     checkAttributes(attr) {
         if (attr === undefined) {
@@ -78136,7 +78317,7 @@ Make sure to test the downported code, it might not always be completely correct
         if (found) {
             return found;
         }
-        found = this.outlineGetReferenceSimple(high, lowFile);
+        found = this.outlineGetReferenceSimple(high, lowFile, highSyntax);
         if (found) {
             return found;
         }
@@ -78811,7 +78992,17 @@ ${indentation}CATCH ${className} INTO ${targetName}.`;
         const fix = edit_helper_1.EditHelper.replaceRange(lowFile, node.getStart(), node.getEnd(), code);
         return issue_1.Issue.atToken(lowFile, node.getFirstToken(), "Outline DATA", this.getMetadata().key, this.conf.severity, fix);
     }
-    outlineGetReferenceSimple(node, lowFile) {
+    /** the type of a reference to the source: LIKE REF TO it, or REF TO data when its type is generic,
+     *  as LIKE REF TO a generically typed field symbol or parameter does not activate */
+    refTypeOf(sourceName, node, lowFile, highSyntax) {
+        var _a;
+        const spag = highSyntax.spaghetti.lookupPosition(node.getFirstToken().getStart(), lowFile.getFilename());
+        if (((_a = spag === null || spag === void 0 ? void 0 : spag.findVariable(sourceName)) === null || _a === void 0 ? void 0 : _a.getType().isGeneric()) === true) {
+            return "TYPE REF TO data";
+        }
+        return "LIKE REF TO " + sourceName;
+    }
+    outlineGetReferenceSimple(node, lowFile, highSyntax) {
         var _a, _b, _c;
         if (!(node.get() instanceof Statements.GetReference)) {
             return undefined;
@@ -78828,7 +79019,8 @@ ${indentation}CATCH ${className} INTO ${targetName}.`;
         const indentation = " ".repeat(node.getFirstToken().getStart().getCol() - 1);
         const firstToken = target.getFirstToken();
         const lastToken = target.getLastToken();
-        const fix1 = edit_helper_1.EditHelper.insertAt(lowFile, node.getStart(), `DATA ${targetName} LIKE REF TO ${source.concatTokens()}.\n${indentation}`);
+        const refType = this.refTypeOf(source.concatTokens(), node, lowFile, highSyntax);
+        const fix1 = edit_helper_1.EditHelper.insertAt(lowFile, node.getStart(), `DATA ${targetName} ${refType}.\n${indentation}`);
         const fix2 = edit_helper_1.EditHelper.replaceRange(lowFile, firstToken.getStart(), lastToken.getEnd(), targetName);
         const fix = edit_helper_1.EditHelper.merge(fix2, fix1);
         return issue_1.Issue.atToken(lowFile, node.getFirstToken(), "Outline DATA", this.getMetadata().key, this.conf.severity, fix);
@@ -78845,7 +79037,7 @@ ${indentation}CATCH ${className} INTO ${targetName}.`;
         if (!(((_a = target === null || target === void 0 ? void 0 : target.getFirstChild()) === null || _a === void 0 ? void 0 : _a.get()) instanceof Expressions.InlineData)) {
             return undefined;
         }
-        let type = "";
+        let type;
         const source = node.findFirstExpression(Expressions.Source);
         if (source === undefined) {
             return undefined;
@@ -78935,8 +79127,8 @@ ${indentation}CATCH ${className} INTO ${targetName}.`;
         if (!(node.get() instanceof Statements.Raise)) {
             return undefined;
         }
-        let id = undefined;
-        let number = undefined;
+        let id;
+        let number;
         let startToken = node.findDirectTokenByText("ID");
         if (startToken) {
             const sources = node.findDirectExpressions(Expressions.Source);
@@ -79094,7 +79286,7 @@ ${indentation}${uniqueName2}->if_t100_message~t100key = ${uniqueName1}.\n`;
             || sourceRef.getChildren().length !== 1) {
             return undefined;
         }
-        let code = "";
+        let code;
         if (sourceRef.findFirstExpression(Expressions.TableExpression)) {
             const uniqueName = this.uniqueName(high.getFirstToken().getStart(), lowFile.getFilename(), highSyntax);
             code = `ASSIGN ${sourceRef.concatTokens()} TO FIELD-SYMBOL(<${uniqueName}>).
@@ -79286,7 +79478,7 @@ LOOP AT ${groupTargetName}tab ${groupTarget}.`;
             || !(tableExpression instanceof nodes_1.ExpressionNode)) {
             return undefined;
         }
-        let condition = "";
+        let condition;
         if (tableExpression.getChildren().length === 3) {
             const index = tableExpression.findDirectExpression(Expressions.Source);
             if (index === undefined) {
@@ -79470,7 +79662,7 @@ ${indentation}${uniqueName}`;
             return undefined;
         }
         const op = secondChild.getFirstToken();
-        let operator = "";
+        let operator;
         switch (op.getStr()) {
             case "+":
                 operator = " + ";
@@ -79512,7 +79704,7 @@ ${indentation}${uniqueName}`;
                 || (formatting === null || formatting === void 0 ? void 0 : formatting.startsWith("ALPHA = ")) === false) {
                 continue;
             }
-            let functionName = "";
+            let functionName;
             switch (formatting) {
                 case "ALPHA = IN":
                     functionName = "CONVERSION_EXIT_ALPHA_INPUT";
@@ -79808,13 +80000,12 @@ ${indentation}    output = ${uniqueName}.\n`;
             const uniqueName = this.uniqueName(firstToken.getStart(), lowFile.getFilename(), highSyntax);
             const indentation = " ".repeat(high.getFirstToken().getStart().getCol() - 1);
             let body = "";
-            let name = "";
             const switchBody = i.findDirectExpression(Expressions.SwitchBody);
             if (switchBody === undefined) {
                 continue;
             }
             for (const l of ((_c = switchBody === null || switchBody === void 0 ? void 0 : switchBody.findDirectExpression(Expressions.Let)) === null || _c === void 0 ? void 0 : _c.findDirectExpressions(Expressions.InlineFieldDefinition)) || []) {
-                name = l.getFirstToken().getStr();
+                const name = l.getFirstToken().getStr();
                 body += indentation + `DATA(${name}) = ${(_d = switchBody.findFirstExpression(Expressions.Source)) === null || _d === void 0 ? void 0 : _d.concatTokens()}.\n`;
             }
             body += `DATA ${uniqueName} ${type}.\n`;
@@ -79872,7 +80063,6 @@ ${indentation}    output = ${uniqueName}.\n`;
             const uniqueName = this.uniqueName(firstToken.getStart(), lowFile.getFilename(), highSyntax);
             const indentation = " ".repeat(high.getFirstToken().getStart().getCol() - 1);
             let body = "";
-            let name = "";
             const reduceBody = i.findDirectExpression(Expressions.ReduceBody);
             if (reduceBody === undefined) {
                 continue;
@@ -79883,7 +80073,7 @@ ${indentation}    output = ${uniqueName}.\n`;
             }
             let firstName = "";
             for (const init of reduceBody.findDirectExpressions(Expressions.InlineFieldDefinition)) {
-                name = init.getFirstToken().getStr();
+                const name = init.getFirstToken().getStr();
                 if (firstName === "") {
                     firstName = name;
                 }
@@ -80115,7 +80305,7 @@ ${indentation}    output = ${uniqueName}.\n`;
                         after += c.concatTokens();
                     }
                 }
-                let condition = "";
+                let condition;
                 if ((tableExpression === null || tableExpression === void 0 ? void 0 : tableExpression.getChildren().length) === 3) {
                     condition = "INDEX " + ((_d = tableExpression === null || tableExpression === void 0 ? void 0 : tableExpression.findDirectExpression(Expressions.Source)) === null || _d === void 0 ? void 0 : _d.concatTokens());
                 }
@@ -80266,7 +80456,7 @@ ${indentation}    output = ${uniqueName}.\n`;
                 continue;
             }
             const name = nameToken.getStr();
-            let type = "";
+            let type;
             if (high.concatTokens().toUpperCase().startsWith("APPEND INITIAL LINE TO ")) {
                 type = "LIKE LINE OF " + ((_b = high.findFirstExpression(Expressions.Target)) === null || _b === void 0 ? void 0 : _b.concatTokens());
             }
@@ -80632,7 +80822,7 @@ ${indentation}    output = ${uniqueName}.\n`;
         }
         return undefined;
     }
-    getReference(node, lowFile, _highSyntax) {
+    getReference(node, lowFile, highSyntax) {
         var _a, _b, _c;
         if (!(node.get() instanceof Statements.GetReference)) {
             return undefined;
@@ -80646,7 +80836,7 @@ ${indentation}    output = ${uniqueName}.\n`;
         if (targetName === undefined || sourceName === undefined) {
             return undefined;
         }
-        const code = `DATA ${targetName} LIKE REF TO ${sourceName}.\n`;
+        const code = `DATA ${targetName} ${this.refTypeOf(sourceName, node, lowFile, highSyntax)}.\n`;
         const fix1 = edit_helper_1.EditHelper.insertAt(lowFile, node.getFirstToken().getStart(), code);
         const fix2 = edit_helper_1.EditHelper.replaceRange(lowFile, inline.getFirstToken().getStart(), inline.getLastToken().getEnd(), targetName);
         const fix = edit_helper_1.EditHelper.merge(fix2, fix1);
@@ -80723,18 +80913,30 @@ ${indentation}    output = ${uniqueName}.\n`;
                 const uniqueName = this.uniqueName(node.getFirstToken().getStart(), lowFile.getFilename(), highSyntax);
                 const indentation = " ".repeat(node.getFirstToken().getStart().getCol() - 1);
                 const sy = func === "LINE_EXISTS" ? "sy-subrc" : "sy-tabix";
-                const code = `DATA ${uniqueName} LIKE sy-subrc.\n` +
-                    indentation + `READ TABLE ${tableName} ${condition}TRANSPORTING NO FIELDS.\n` +
-                    indentation + uniqueName + ` = ${sy}.\n` +
-                    indentation;
                 let insertAt = node.getFirstToken().getStart();
                 if (node.get() instanceof statements_1.ElseIf) {
-                    // assumption: no side effects in IF conditions
+                    // assumption: no side effects in IF conditions, except the READ itself, see below
                     insertAt = this.findStartOfIf(node, highFile);
                     if (insertAt === undefined) {
                         continue;
                     }
                 }
+                // the READ sets sy-subrc and sy-tabix, which the built-in function does not,
+                // keep the values for the code that reads them after the inserted READ
+                const keep = [];
+                const reads = this.readSyFields(insertAt, highFile);
+                for (const field of ["sy-subrc", "sy-tabix"]) {
+                    if (reads.has(field)) {
+                        keep.push({ field, name: this.uniqueName(node.getFirstToken().getStart(), lowFile.getFilename(), highSyntax) });
+                    }
+                }
+                const code = `DATA ${uniqueName} LIKE sy-subrc.\n` +
+                    keep.map(k => indentation + `DATA ${k.name} LIKE ${k.field}.\n`).join("") +
+                    keep.map(k => indentation + `${k.name} = ${k.field}.\n`).join("") +
+                    indentation + `READ TABLE ${tableName} ${condition}TRANSPORTING NO FIELDS.\n` +
+                    indentation + uniqueName + ` = ${sy}.\n` +
+                    keep.map(k => indentation + `${k.field} = ${k.name}.\n`).join("") +
+                    indentation;
                 const fix1 = edit_helper_1.EditHelper.insertAt(lowFile, insertAt, code);
                 const start = expression.getFirstToken().getStart();
                 const end = expression.getLastToken().getEnd();
@@ -80745,6 +80947,46 @@ ${indentation}    output = ${uniqueName}.\n`;
         }
         return undefined;
     }
+    /** sy-subrc and sy-tabix, when read from the position on, in the procedure around it or the file outside of one;
+     *  inside a loop from the start of the outermost loop, as the next pass reads what this one left */
+    readSyFields(position, highFile) {
+        const structure = highFile.getStructure();
+        const contains = (s) => !position.isBefore(s.getFirstToken().getStart())
+            && !position.isAfter(s.getLastToken().getStart());
+        let scope = undefined;
+        for (const s of [Structures.Method, Structures.Form, Structures.FunctionModule]) {
+            for (const p of (structure === null || structure === void 0 ? void 0 : structure.findAllStructuresRecursive(s)) || []) {
+                if (contains(p)) {
+                    scope = p;
+                }
+            }
+        }
+        scope = scope || structure;
+        let from = position;
+        for (const s of [Structures.Loop, Structures.Do, Structures.While, Structures.Select]) {
+            for (const l of (scope === null || scope === void 0 ? void 0 : scope.findAllStructuresRecursive(s)) || []) {
+                if (contains(l) && l.getFirstToken().getStart().isBefore(from)) {
+                    from = l.getFirstToken().getStart();
+                }
+            }
+        }
+        const ret = new Set();
+        for (const s of (scope === null || scope === void 0 ? void 0 : scope.findAllStatementNodes()) || []) {
+            if (s.getLastToken().getStart().isBefore(from)) {
+                continue;
+            }
+            const tokens = s.getTokens();
+            for (let i = 0; i < tokens.length - 2; i++) {
+                if (tokens[i].getStr().toUpperCase() === "SY" && tokens[i + 1].getStr() === "-") {
+                    const field = "sy-" + tokens[i + 2].getStr().toLowerCase();
+                    if (field === "sy-subrc" || field === "sy-tabix") {
+                        ret.add(field);
+                    }
+                }
+            }
+        }
+        return ret;
+    }
     findStartOfIf(node, highFile) {
         const structure = highFile.getStructure();
         for (const c of (structure === null || structure === void 0 ? void 0 : structure.findAllStructuresRecursive(Structures.If)) || []) {
@@ -80757,6 +80999,7 @@ ${indentation}    output = ${uniqueName}.\n`;
         return undefined;
     }
     newToCreateObject(low, high, lowFile, highSyntax) {
+        var _a;
         if (!(low.get() instanceof _statement_1.Unknown)) {
             return undefined;
         }
@@ -80772,7 +81015,7 @@ ${indentation}    output = ${uniqueName}.\n`;
                 && found
                 && source.concatTokens() === found.concatTokens()
                 && target.findDirectExpression(Expressions.InlineData) === undefined) {
-                const abap = this.newParameters(found, target.concatTokens(), highSyntax, lowFile);
+                const abap = (_a = this.newDataReference(high, target, found, lowFile, highSyntax)) !== null && _a !== void 0 ? _a : this.newParameters(found, target.concatTokens(), highSyntax, lowFile);
                 if (abap !== undefined) {
                     fix = edit_helper_1.EditHelper.replaceRange(lowFile, high.getFirstToken().getStart(), high.getLastToken().getEnd(), abap);
                 }
@@ -80811,6 +81054,42 @@ ${indentation}    output = ${uniqueName}.\n`;
             return undefined;
         }
     }
+    /** NEW into a data reference creates data, CREATE OBJECT cannot: "target = NEW #|type( [value] )"
+     *  with a variable typed REF TO a data type becomes CREATE DATA plus the assignment of the value */
+    newDataReference(high, target, found, lowFile, highSyntax) {
+        var _a, _b, _c;
+        if (target.getChildren().length !== 1 || !(((_a = target.getFirstChild()) === null || _a === void 0 ? void 0 : _a.get()) instanceof Expressions.TargetField)) {
+            return undefined;
+        }
+        const children = found.getChildren();
+        const value = found.findDirectExpression(Expressions.Source);
+        // NEW, the type, "(", optionally one value, ")"
+        if (children.length !== 4 + (value ? 1 : 0)) {
+            return undefined;
+        }
+        const name = target.concatTokens();
+        const spag = highSyntax.spaghetti.lookupPosition(high.getFirstToken().getStart(), lowFile.getFilename());
+        const type = (_b = spag === null || spag === void 0 ? void 0 : spag.findVariable(name)) === null || _b === void 0 ? void 0 : _b.getType();
+        if (!(type instanceof basic_1.DataReference)) {
+            return undefined;
+        }
+        const typeName = (_c = found.findDirectExpression(Expressions.TypeNameOrInfer)) === null || _c === void 0 ? void 0 : _c.concatTokens();
+        const indentation = " ".repeat(high.getFirstToken().getStart().getCol() - 1);
+        let abap = `CREATE DATA ${name}` + (typeName === "#" ? "" : ` TYPE ${typeName}`) + ".";
+        if (value === undefined) {
+            return abap;
+        }
+        else if (type.getType().isGeneric() === false) {
+            return abap + `\n${indentation}${name}->* = ${value.concatTokens()}.`;
+        }
+        // a generic reference cannot be dereferenced as a target below v756
+        const fs = this.uniqueName(high.getFirstToken().getStart(), lowFile.getFilename(), highSyntax, true);
+        abap = `FIELD-SYMBOLS ${fs} TYPE any.\n` +
+            indentation + abap + "\n" +
+            indentation + `ASSIGN ${name}->* TO ${fs}.\n` +
+            indentation + `${fs} = ${value.concatTokens()}.`;
+        return abap;
+    }
     newParameters(found, name, highSyntax, lowFile) {
         var _a, _b, _c;
         const typeToken = (_a = found.findDirectExpression(Expressions.TypeNameOrInfer)) === null || _a === void 0 ? void 0 : _a.getFirstToken();
@@ -81602,6 +81881,8 @@ class EmptyStructureConf extends _basic_rule_config_1.BasicRuleConfig {
         this.try = true;
         /** Checks for empty WHEN blocks */
         this.when = true;
+        /** Checks for empty CATCH blocks, unless the CATCH has pragma ##NO_HANDLER or pseudo comment "#EC NO_HANDLER */
+        this.catch = false;
         // todo, other category containing ELSE
     }
 }
@@ -81714,6 +81995,21 @@ result = xsdbool( sy-subrc = 0 ).`,
                 }
             }
         }
+        if (this.getConfig().catch === true) {
+            for (const c of stru.findAllStructuresRecursive(Structures.Catch)) {
+                // a comment in the block does not count, the extended check (SLIN) reports it too
+                if (c.findDirectStructure(Structures.Body) !== undefined) {
+                    continue;
+                }
+                const catchStatement = c.getFirstStatement();
+                if (catchStatement === undefined || this.isNoHandler(catchStatement, statements)) {
+                    continue;
+                }
+                const token = c.getFirstToken();
+                const issue = issue_1.Issue.atToken(file, token, this.getDescription(c.get().constructor.name), this.getMetadata().key, this.conf.severity);
+                issues.push(issue);
+            }
+        }
         if (this.getConfig().when === true) {
             const tries = stru.findAllStructuresRecursive(Structures.When);
             for (const t of tries) {
@@ -81727,6 +82023,15 @@ result = xsdbool( sy-subrc = 0 ).`,
         }
         return issues;
     }
+    isNoHandler(catchStatement, statements) {
+        if (catchStatement.getPragmas().some(p => p.getStr().toUpperCase() === "##NO_HANDLER")) {
+            return true;
+        }
+        const next = statements[statements.indexOf(catchStatement) + 1];
+        return (next === null || next === void 0 ? void 0 : next.get()) instanceof _statement_1.Comment
+            && next.getStart().getRow() === catchStatement.getEnd().getRow()
+            && next.concatTokens().toUpperCase().replace(/\s/g, "").includes("\"#ECNO_HANDLER");
+    }
 }
 exports.EmptyStructure = EmptyStructure;
 //# sourceMappingURL=empty_structure.js.map
@@ -83163,7 +83468,7 @@ cl_abap_typedescr=>describe_by_name(
         const methodSource = statNode.findDirectExpression(Expressions.MethodSource);
         let methodSourceStr = methodSource === null || methodSource === void 0 ? void 0 : methodSource.concatTokens();
         const methodBody = statNode.findDirectExpression(Expressions.MethodCallBody);
-        let methodBodyStr = "";
+        let methodBodyStr;
         if (methodBody) {
             const methodCallParam = methodBody.findDirectExpression(Expressions.MethodCallParam);
             if (methodCallParam && methodCallParam.getFirstToken().getStr() === "(") {
@@ -86057,7 +86362,7 @@ class KeywordCase extends _abap_rule_1.ABAPRule {
         const firstToken = tokens[0].token;
         const lastToken = tokens[tokens.length - 1].token;
         const firstTokenValue = firstToken.getStr();
-        let description = "";
+        let description;
         if (first.keyword === true) {
             description = `Keyword should be ${this.conf.style} case: "${firstTokenValue}"`;
         }
@@ -86582,7 +86887,7 @@ ENDCLASS.`,
                 continue;
             }
             const className = classDef.name;
-            let expected = "";
+            let expected;
             if (classDef.isForTesting) {
                 expected = this.conf.test;
             }
@@ -87248,7 +87553,7 @@ ENDIF.
             issues.push(...this.analyzeCondition(file, cond));
         }
         for (const sub of structure.findAllExpressionsMulti([Expressions.CondSub, Expressions.ComponentCondSub])) {
-            let cond = [];
+            let cond;
             if (sub.get() instanceof Expressions.CondSub) {
                 cond = sub.findDirectExpressions(Expressions.Cond);
             }
@@ -91286,13 +91591,9 @@ ENDIF.`,
         }
         else if (statement instanceof Statements.ClassDefinitionLoad ||
             statement instanceof Statements.InterfaceLoad) {
-            let token = undefined;
-            if (statement instanceof Statements.ClassDefinitionLoad) {
-                token = statementNode.getChildren()[3].getFirstToken();
-            }
-            else {
-                token = statementNode.getChildren()[2].getFirstToken();
-            }
+            const token = statement instanceof Statements.ClassDefinitionLoad
+                ? statementNode.getChildren()[3].getFirstToken()
+                : statementNode.getChildren()[2].getFirstToken();
             let startPosition = token.getStart();
             startPosition = new position_1.Position(startPosition.getRow(), startPosition.getCol() - 1);
             return edit_helper_1.EditHelper.deleteRange(file, startPosition, token.getEnd());
@@ -92089,7 +92390,6 @@ Object.defineProperty(exports, "__esModule", ({ value: true }));
 exports.ParserMissingSpace = exports.ParserMissingSpaceConf = void 0;
 const Expressions = __importStar(__webpack_require__(/*! ../abap/2_statements/expressions */ "../core/build/src/abap/2_statements/expressions/index.js"));
 const issue_1 = __webpack_require__(/*! ../issue */ "../core/build/src/issue.js");
-const position_1 = __webpack_require__(/*! ../position */ "../core/build/src/position.js");
 const _abap_rule_1 = __webpack_require__(/*! ./_abap_rule */ "../core/build/src/rules/_abap_rule.js");
 const nodes_1 = __webpack_require__(/*! ../abap/nodes */ "../core/build/src/abap/nodes/index.js");
 const _basic_rule_config_1 = __webpack_require__(/*! ./_basic_rule_config */ "../core/build/src/rules/_basic_rule_config.js");
@@ -92122,7 +92422,7 @@ This rule makes sure the spaces are consistently required across the language.`,
     }
     runParsed(file) {
         const issues = [];
-        let start = new position_1.Position(0, 0);
+        let start;
         for (const statement of file.getStatements()) {
             const missing = this.missingSpace(statement);
             if (missing) {
@@ -93683,14 +93983,8 @@ From 752 and up`,
     }
     getFix(file, statement, withExporting) {
         const children = statement.getChildren();
-        let contentFix = undefined;
-        if (withExporting) {
-            const fixText = "( " + children[5].concatTokens() + " ).";
-            contentFix = edit_helper_1.EditHelper.replaceRange(file, children[3].getLastToken().getEnd(), statement.getEnd(), fixText);
-        }
-        else {
-            contentFix = edit_helper_1.EditHelper.replaceRange(file, children[3].getLastToken().getEnd(), statement.getEnd(), "( ).");
-        }
+        const fixText = withExporting ? "( " + children[5].concatTokens() + " )." : "( ).";
+        const contentFix = edit_helper_1.EditHelper.replaceRange(file, children[3].getLastToken().getEnd(), statement.getEnd(), fixText);
         const replaceType = edit_helper_1.EditHelper.replaceToken(file, children[2].getFirstToken(), "NEW");
         return edit_helper_1.EditHelper.merge(contentFix, replaceType);
     }
@@ -94924,7 +95218,8 @@ class RedundantConversion {
             key: "redundant_conversion",
             title: "Redundant Conversion",
             shortDescription: `Find redundant CONV expressions`,
-            extendedInformation: `Reports CONV expressions whose operand already has the conversion's target type.`,
+            extendedInformation: `Reports CONV expressions whose operand already has the conversion's target type,
+including the result of a built-in function with a fixed result type, like to_upper( ) or strlen( ).`,
             tags: [_irule_1.RuleTag.Quickfix],
             badExample: `DATA text TYPE string.
 text = CONV string( text ).`,
@@ -94961,7 +95256,8 @@ text = text.`,
             if (structure === undefined) {
                 continue;
             }
-            for (const source of structure.findAllExpressions(Expressions.Source)) {
+            // recursive, a CONV can be the argument of a built-in function, which is itself a Source
+            for (const source of structure.findAllExpressionsRecursive(Expressions.Source)) {
                 if (source.getFirstToken().getStr().toUpperCase() !== "CONV") {
                     continue;
                 }
@@ -95020,6 +95316,15 @@ text = text.`,
     }
     sourceType(source, scope) {
         let context;
+        // boolc( <condition> ) is not a MethodCallChain, xsdbool( ) has the same shape but is not reported
+        const first = source.getFirstToken();
+        if (first.getStr().toUpperCase() === "BOOLC"
+            && source.findDirectExpression(Expressions.Cond) !== undefined
+            && source.getChildren().length === 4
+            && scope.getData().references.some(reference => reference.referenceType === _reference_1.ReferenceType.BuiltinMethodReference
+                && reference.position.getStart().equals(first.getStart()))) {
+            return basic_1.StringType.get();
+        }
         for (const child of source.getChildren()) {
             if (!(child instanceof nodes_1.ExpressionNode)) {
                 continue;
@@ -95030,6 +95335,15 @@ text = text.`,
                     return undefined;
                 }
             }
+            else if (child.get() instanceof Expressions.MethodCallChain) {
+                if (source.getChildren().length !== 1) {
+                    return undefined;
+                }
+                context = this.builtinResultType(child, scope);
+                if (context === undefined) {
+                    return undefined;
+                }
+            }
             else if (child.get() instanceof Expressions.Constant) {
                 context = this.infer(context, constant_1.Constant.runSyntax(child));
             }
@@ -95050,6 +95364,32 @@ text = text.`,
         }
         return context;
     }
+    /** The result type of a call to a built-in function whose result type does not depend on its arguments,
+     *  undefined for anything else, e.g. abs( ) or xsdbool( ), or a method of the same name */
+    builtinResultType(chain, scope) {
+        var _a;
+        const children = chain.getChildren();
+        if (children.length !== 1 || !(children[0].get() instanceof Expressions.MethodCall)) {
+            return undefined;
+        }
+        const name = (_a = children[0].findDirectExpression(Expressions.MethodName)) === null || _a === void 0 ? void 0 : _a.getFirstToken();
+        if (name === undefined) {
+            return undefined;
+        }
+        const isBuiltin = scope.getData().references.some(reference => reference.referenceType === _reference_1.ReferenceType.BuiltinMethodReference
+            && reference.position.getStart().equals(name.getStart()));
+        if (isBuiltin === false) {
+            return undefined;
+        }
+        const upper = name.getStr().toUpperCase();
+        if (RedundantConversion.stringResult.has(upper)) {
+            return basic_1.StringType.get();
+        }
+        else if (RedundantConversion.integerResult.has(upper)) {
+            return basic_1.IntegerType.get();
+        }
+        return undefined;
+    }
     fieldChainType(fieldChain, scope) {
         var _a;
         const operand = (_a = scope.getData().references.find(reference => reference.referenceType === _reference_1.ReferenceType.DataReadReference
@@ -95114,6 +95454,17 @@ text = text.`,
     }
 }
 exports.RedundantConversion = RedundantConversion;
+// built-in functions that always return a string, whatever their arguments
+RedundantConversion.stringResult = new Set([
+    "BOOLC", "CONCAT_LINES_OF", "CONDENSE", "ESCAPE", "FROM_MIXED", "INSERT", "REPEAT", "REPLACE",
+    "REVERSE", "SEGMENT", "SHIFT_LEFT", "SHIFT_RIGHT", "SUBSTRING", "SUBSTRING_AFTER", "SUBSTRING_BEFORE",
+    "SUBSTRING_FROM", "SUBSTRING_TO", "TO_LOWER", "TO_MIXED", "TO_UPPER", "TRANSLATE"
+]);
+// built-in functions that always return an i
+RedundantConversion.integerResult = new Set([
+    "CHARLEN", "COUNT", "COUNT_ANY_NOT_OF", "COUNT_ANY_OF", "DBMAXLEN", "DISTANCE", "FIND", "FIND_ANY_NOT_OF",
+    "FIND_ANY_OF", "FIND_END", "LINES", "NUMOFCHAR", "STRLEN", "XSTRLEN"
+]);
 //# sourceMappingURL=redundant_conversion.js.map
 
 /***/ },
@@ -102060,6 +102411,7 @@ class XMLConsistency {
 * Naming for CLAS and INTF objects
 * QUAN fields in TABL objects have reference table and field values
 * Transparent TABL objects have delivery class and size category set
+* TABL fields with a built-in type carry the INTTYPE abapGit serializes for it, for the confirmed pairs
 * Lock parameter names in ENQU objects are max 16 characters
 * Texts and translations do not exceed maximum allowed length.`,
             tags: [_irule_1.RuleTag.Naming, _irule_1.RuleTag.Syntax],
@@ -102319,15 +102671,29 @@ class XMLConsistency {
     }
     runTable(obj, file) {
         var _a, _b, _c, _d;
-        var _e;
+        var _e, _f;
         const issues = [];
         for (const field of (_e = obj.getFields()) !== null && _e !== void 0 ? _e : []) {
             if (field.DATATYPE === "QUAN" && (!((_a = field.REFTABLE) === null || _a === void 0 ? void 0 : _a.trim()) || !((_b = field.REFFIELD) === null || _b === void 0 ? void 0 : _b.trim()))) {
                 const message = `QUAN field ${field.FIELDNAME} must have REFTABLE and REFFIELD set`;
                 issues.push(issue_1.Issue.atRow(file, 1, message, this.getMetadata().key, this.conf.severity));
             }
+            const expected = field.DATATYPE ? XMLConsistency.intTypes[field.DATATYPE] : undefined;
+            if (expected !== undefined && field.INTTYPE !== undefined && field.INTTYPE !== expected) {
+                const message = `Field ${field.FIELDNAME}: INTTYPE "${field.INTTYPE}" does not match DATATYPE ${field.DATATYPE}, ` +
+                    `abapGit serializes "${expected}"`;
+                issues.push(issue_1.Issue.atRow(file, 1, message, this.getMetadata().key, this.conf.severity));
+            }
         }
         if (obj.getTableCategory() === Objects.TableCategory.Transparent) {
+            // Confirmed reserved database field names, see issue #4331, DAY and HOUR reported by activation.
+            // TRESE also contains names such as TEXT and LENGTH which are allowed here.
+            const reservedFieldNames = ["ZONE", "HANDLER", "SECTION", "PARAMETER", "DAY", "HOUR"];
+            for (const field of (_f = obj.getFields()) !== null && _f !== void 0 ? _f : []) {
+                if (reservedFieldNames.includes(field.FIELDNAME.toUpperCase())) {
+                    issues.push(issue_1.Issue.atRow(file, 1, `Table field name "${field.FIELDNAME}" is reserved`, this.getMetadata().key, this.conf.severity));
+                }
+            }
             if (!((_c = obj.getDeliveryClass()) === null || _c === void 0 ? void 0 : _c.trim())) {
                 const message = `Transparent table must have delivery class(CONTFLAG in DD02V) set`;
                 issues.push(issue_1.Issue.atRow(file, 1, message, this.getMetadata().key, this.conf.transparentTableSeverity));
@@ -102341,6 +102707,15 @@ class XMLConsistency {
     }
 }
 exports.XMLConsistency = XMLConsistency;
+// INTTYPE that abapGit serializes for a built-in DATATYPE, confirmed by exported tables;
+// a different value activates, but shows as a diff on every pull
+XMLConsistency.intTypes = {
+    "CHAR": "C", "CLNT": "C", "CUKY": "C", "LANG": "C",
+    "DATS": "D", "TIMS": "T",
+    "INT4": "X", "RAW": "X",
+    "DEC": "P", "CURR": "P",
+    "STRG": "g", "SSTR": "g", "RSTR": "y",
+};
 //# sourceMappingURL=xml_consistency.js.map
 
 /***/ },
@@ -102963,7 +103338,7 @@ class FormLengthStats {
         return res;
     }
     static findName(stat) {
-        let name = "";
+        let name;
         const nameExpr = stat.findFirstExpression(expressions_1.FormName);
         if (nameExpr) {
             name = nameExpr.getFirstToken().getStr();
@@ -103374,7 +103749,7 @@ class MethodLengthStats {
         return res;
     }
     static findName(stat) {
-        let name = "";
+        let name;
         const nameExpr = stat.findFirstExpression(expressions_1.MethodName);
         if (nameExpr) {
             name = nameExpr.getFirstToken().getStr();
@@ -104976,7 +105351,7 @@ function loadConfig(filename) {
     // c) specified and found
     // d) specified and not found => use default
     // e) supplied but a directory => use default
-    let f = "";
+    let f;
     if (filename === undefined) {
         f = process.cwd() + path.sep + "abaplint.json";
         if (fs.existsSync(f) === false) {
@@ -105119,7 +105494,7 @@ async function run(arg) {
     else {
         process.stderr.write("abaplint " + core_1.Registry.abaplintVersion() + "\n");
         let loaded = [];
-        let deps = [];
+        let deps;
         const { config, base } = loadConfig(arg.configFilename);
         outputSyntaxConfig(config);
         try {