@abaplint/cli 2.119.67 → 2.120.1

package/build/cli.js

@@ -26002,7 +26002,7 @@ class CurrentScope {
         return this.reg.getConfig().getLanguageVersion();
     }
     getOpenABAP() {
-        return this.reg.getConfig().getOpenABAP();
+        return this.reg.getConfig().isOpenABAP();
     }
     getRegistry() {
         return this.reg;
@@ -29006,6 +29006,75 @@ exports.BasicTypes = BasicTypes;
 
 /***/ },
 
+/***/ "../core/build/src/abap/5_syntax/expressions/_check_offset_length.js"
+/*!***************************************************************************!*\
+  !*** ../core/build/src/abap/5_syntax/expressions/_check_offset_length.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.checkOffsetLength = checkOffsetLength;
+const Expressions = __importStar(__webpack_require__(/*! ../../2_statements/expressions */ "../core/build/src/abap/2_statements/expressions/index.js"));
+const basic_1 = __webpack_require__(/*! ../../types/basic */ "../core/build/src/abap/types/basic/index.js");
+const _syntax_input_1 = __webpack_require__(/*! ../_syntax_input */ "../core/build/src/abap/5_syntax/_syntax_input.js");
+const _builtin_1 = __webpack_require__(/*! ../_builtin */ "../core/build/src/abap/5_syntax/_builtin.js");
+// Offsets or lengths cannot be specified for STRING or XSTRING fields
+// passed as a method parameter source. Returns true if an issue was reported.
+function checkOffsetLength(source, sourceType, method, input) {
+    const chain = source.findDirectExpression(Expressions.FieldChain);
+    const isCalculated = source.findDirectExpression(Expressions.Source) !== undefined;
+    const hasOffsetOrLength = chain !== undefined
+        && isCalculated === false
+        && (chain.findDirectExpression(Expressions.FieldOffset) !== undefined
+            || chain.findDirectExpression(Expressions.FieldLength) !== undefined);
+    if (hasOffsetOrLength
+        && !(method instanceof _builtin_1.BuiltInMethod)
+        && (sourceType instanceof basic_1.StringType || sourceType instanceof basic_1.XStringType)) {
+        const message = `Offsets or lengths cannot be specified for fields of type "STRING" or "XSTRING" in the current statement`;
+        input.issues.push((0, _syntax_input_1.syntaxIssue)(input, source.getFirstToken(), message));
+        return true;
+    }
+    return false;
+}
+//# sourceMappingURL=_check_offset_length.js.map
+
+/***/ },
+
 /***/ "../core/build/src/abap/5_syntax/expressions/attribute_chain.js"
 /*!**********************************************************************!*\
   !*** ../core/build/src/abap/5_syntax/expressions/attribute_chain.js ***!
@@ -32150,7 +32219,7 @@ const source_1 = __webpack_require__(/*! ./source */ "../core/build/src/abap/5_s
 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 constant_1 = __webpack_require__(/*! ./constant */ "../core/build/src/abap/5_syntax/expressions/constant.js");
-const _builtin_1 = __webpack_require__(/*! ../_builtin */ "../core/build/src/abap/5_syntax/_builtin.js");
+const _check_offset_length_1 = __webpack_require__(/*! ./_check_offset_length */ "../core/build/src/abap/5_syntax/expressions/_check_offset_length.js");
 class MethodCallParam {
     static runSyntax(node, input, method) {
         if (!(node.get() instanceof Expressions.MethodCallParam)) {
@@ -32211,17 +32280,7 @@ class MethodCallParam {
             let sourceType = undefined;
             if (child.get() instanceof Expressions.Source) {
                 sourceType = source_1.Source.runSyntax(child, input, targetType);
-                const chain = child.findDirectExpression(Expressions.FieldChain);
-                const isCalculated = child.findDirectExpression(Expressions.Source) !== undefined;
-                const hasOffsetOrLength = chain !== undefined
-                    && isCalculated === false
-                    && (chain.findDirectExpression(Expressions.FieldOffset) !== undefined
-                        || chain.findDirectExpression(Expressions.FieldLength) !== undefined);
-                if (hasOffsetOrLength
-                    && !(method instanceof _builtin_1.BuiltInMethod)
-                    && (sourceType instanceof basic_1.StringType || sourceType instanceof basic_1.XStringType)) {
-                    const message = `Offsets or lengths cannot be specified for fields of type "STRING" or "XSTRING" in the current statement`;
-                    input.issues.push((0, _syntax_input_1.syntaxIssue)(input, child.getFirstToken(), message));
+                if ((0, _check_offset_length_1.checkOffsetLength)(child, sourceType, method, input)) {
                     return;
                 }
             }
@@ -32480,6 +32539,7 @@ const _syntax_input_1 = __webpack_require__(/*! ../_syntax_input */ "../core/bui
 const assert_error_1 = __webpack_require__(/*! ../assert_error */ "../core/build/src/abap/5_syntax/assert_error.js");
 const field_chain_1 = __webpack_require__(/*! ./field_chain */ "../core/build/src/abap/5_syntax/expressions/field_chain.js");
 const _reference_1 = __webpack_require__(/*! ../_reference */ "../core/build/src/abap/5_syntax/_reference.js");
+const _check_offset_length_1 = __webpack_require__(/*! ./_check_offset_length */ "../core/build/src/abap/5_syntax/expressions/_check_offset_length.js");
 class MethodParameters {
     constructor() {
         this.requiredParameters = undefined;
@@ -32650,7 +32710,6 @@ class MethodParameters {
         }
     }
     parameterListS(node, input, method) {
-        var _a;
         if (node === undefined) {
             return [];
         }
@@ -32658,14 +32717,21 @@ class MethodParameters {
             throw new assert_error_1.AssertError("parameterListS, unexpected node");
         }
         const ret = [];
+        const supplied = new Set();
         for (const c of node.getChildren()) {
             if (!(c.get() instanceof Expressions.ParameterS) || !(c instanceof nodes_1.ExpressionNode)) {
                 throw new assert_error_1.AssertError("parameterListS, unexpected node, child");
             }
-            const name = (_a = c.findDirectExpression(Expressions.ParameterName)) === null || _a === void 0 ? void 0 : _a.getFirstToken().getStr().toUpperCase();
-            if (name === undefined) {
+            const nameNode = c.findDirectExpression(Expressions.ParameterName);
+            const name = nameNode === null || nameNode === void 0 ? void 0 : nameNode.getFirstToken().getStr().toUpperCase();
+            if (name === undefined || nameNode === undefined) {
                 throw new assert_error_1.AssertError("parameterListS, no name determined");
             }
+            if (supplied.has(name)) {
+                const message = `Method parameter "${name}" is supplied more than once`;
+                input.issues.push((0, _syntax_input_1.syntaxIssue)(input, nameNode.getFirstToken(), message));
+            }
+            supplied.add(name);
             const source = c.findDirectExpression(Expressions.Source);
             if (source === undefined) {
                 throw new assert_error_1.AssertError("parameterListS, no source found");
@@ -32683,6 +32749,7 @@ class MethodParameters {
                 targetType = method;
             }
             let sourceType = source_1.Source.runSyntax(source, input, targetType);
+            (0, _check_offset_length_1.checkOffsetLength)(source, sourceType, method, input);
             if (sourceType === undefined) {
                 if (method instanceof basic_1.VoidType) {
                     sourceType = method;
@@ -32698,7 +32765,6 @@ class MethodParameters {
         return ret;
     }
     parameterListT(node, input) {
-        var _a;
         if (node === undefined) {
             return [];
         }
@@ -32706,14 +32772,21 @@ class MethodParameters {
             throw new assert_error_1.AssertError("parameterListT, unexpected node");
         }
         const ret = [];
+        const supplied = new Set();
         for (const c of node.getChildren()) {
             if (!(c.get() instanceof Expressions.ParameterT) || !(c instanceof nodes_1.ExpressionNode)) {
                 throw new assert_error_1.AssertError("parameterListT, unexpected node, child");
             }
-            const name = (_a = c.findDirectExpression(Expressions.ParameterName)) === null || _a === void 0 ? void 0 : _a.getFirstToken().getStr().toUpperCase();
-            if (name === undefined) {
+            const nameNode = c.findDirectExpression(Expressions.ParameterName);
+            const name = nameNode === null || nameNode === void 0 ? void 0 : nameNode.getFirstToken().getStr().toUpperCase();
+            if (name === undefined || nameNode === undefined) {
                 throw new assert_error_1.AssertError("parameterListT, no name determined");
             }
+            if (supplied.has(name)) {
+                const message = `Method parameter "${name}" is supplied more than once`;
+                input.issues.push((0, _syntax_input_1.syntaxIssue)(input, nameNode.getFirstToken(), message));
+            }
+            supplied.add(name);
             const target = c.findDirectExpression(Expressions.Target);
             if (target === undefined) {
                 throw new assert_error_1.AssertError("parameterListT, no target found");
@@ -37242,6 +37315,7 @@ const basic_1 = __webpack_require__(/*! ../../types/basic */ "../core/build/src/
 const target_1 = __webpack_require__(/*! ../expressions/target */ "../core/build/src/abap/5_syntax/expressions/target.js");
 const _reference_1 = __webpack_require__(/*! ../_reference */ "../core/build/src/abap/5_syntax/_reference.js");
 const _syntax_input_1 = __webpack_require__(/*! ../_syntax_input */ "../core/build/src/abap/5_syntax/_syntax_input.js");
+const _type_utils_1 = __webpack_require__(/*! ../_type_utils */ "../core/build/src/abap/5_syntax/_type_utils.js");
 class Catch {
     runSyntax(node, input) {
         var _a, _b;
@@ -37292,7 +37366,22 @@ class Catch {
             }
         }
         else if (target) {
-            target_1.Target.runSyntax(target, input);
+            const targetType = target_1.Target.runSyntax(target, input);
+            if (targetType instanceof basic_1.ObjectReferenceType) {
+                for (const c of node.findDirectExpressions(Expressions.ClassName)) {
+                    const token = c.getFirstToken();
+                    const className = token.getStr().toUpperCase();
+                    const found = input.scope.existsObject(className);
+                    if (found === null || found === void 0 ? void 0 : found.id) {
+                        const catchType = new basic_1.ObjectReferenceType(found.id);
+                        if (new _type_utils_1.TypeUtils(input.scope).isAssignableStrict(catchType, targetType) === false) {
+                            const message = "CATCH target not compatible with " + className;
+                            input.issues.push((0, _syntax_input_1.syntaxIssue)(input, token, message));
+                            return;
+                        }
+                    }
+                }
+            }
         }
     }
 }
@@ -53616,7 +53705,7 @@ class CDSDefineAbstract extends combi_1.Expression {
     getRunnable() {
         const field = (0, combi_1.seq)((0, combi_1.star)(_1.CDSAnnotation), (0, combi_1.optPrio)((0, combi_1.str)("KEY")), cds_name_1.CDSName, ":", _1.CDSType, ";");
         const compsiOrAssoci = (0, combi_1.seq)((0, combi_1.star)(_1.CDSAnnotation), cds_name_1.CDSName, ":", (0, combi_1.alt)(_1.CDSComposition, _1.CDSAssociation), ";");
-        return (0, combi_1.seq)((0, combi_1.star)(_1.CDSAnnotation), "DEFINE", (0, combi_1.opt)("ROOT"), "ABSTRACT", "ENTITY", cds_name_1.CDSName, (0, combi_1.opt)(_1.CDSWithParameters), (0, combi_1.str)("{"), (0, combi_1.plus)((0, combi_1.alt)(field, compsiOrAssoci)), (0, combi_1.str)("}"), (0, combi_1.opt)(";"));
+        return (0, combi_1.seq)((0, combi_1.star)(_1.CDSAnnotation), (0, combi_1.opt)("DEFINE"), (0, combi_1.opt)("ROOT"), "ABSTRACT", "ENTITY", cds_name_1.CDSName, (0, combi_1.opt)(_1.CDSWithParameters), (0, combi_1.str)("{"), (0, combi_1.plus)((0, combi_1.alt)(field, compsiOrAssoci)), (0, combi_1.str)("}"), (0, combi_1.opt)(";"));
     }
 }
 exports.CDSDefineAbstract = CDSDefineAbstract;
@@ -53642,7 +53731,7 @@ class CDSDefineCustom extends combi_1.Expression {
     getRunnable() {
         const field = (0, combi_1.seq)((0, combi_1.optPrio)((0, combi_1.str)("KEY")), cds_name_1.CDSName, ":", cds_type_1.CDSType, ";");
         const compsiOrAssoci = (0, combi_1.seq)(cds_name_1.CDSName, ":", (0, combi_1.alt)(_1.CDSComposition, _1.CDSAssociation), ";");
-        return (0, combi_1.seq)((0, combi_1.star)(_1.CDSAnnotation), (0, combi_1.str)("DEFINE"), (0, combi_1.opt)((0, combi_1.str)("ROOT")), (0, combi_1.str)("CUSTOM ENTITY"), cds_name_1.CDSName, (0, combi_1.opt)(_1.CDSWithParameters), (0, combi_1.str)("{"), (0, combi_1.plus)((0, combi_1.seq)((0, combi_1.star)(_1.CDSAnnotation), (0, combi_1.alt)(field, compsiOrAssoci))), (0, combi_1.str)("}"), (0, combi_1.opt)(";"));
+        return (0, combi_1.seq)((0, combi_1.star)(_1.CDSAnnotation), (0, combi_1.opt)("DEFINE"), (0, combi_1.opt)("ROOT"), (0, combi_1.str)("CUSTOM ENTITY"), cds_name_1.CDSName, (0, combi_1.opt)(_1.CDSWithParameters), (0, combi_1.str)("{"), (0, combi_1.plus)((0, combi_1.seq)((0, combi_1.star)(_1.CDSAnnotation), (0, combi_1.alt)(field, compsiOrAssoci))), (0, combi_1.str)("}"), (0, combi_1.opt)(";"));
     }
 }
 exports.CDSDefineCustom = CDSDefineCustom;
@@ -53672,7 +53761,7 @@ class CDSDefineExternalEntity extends combi_1.Expression {
         const body = (0, combi_1.plus)((0, combi_1.alt)(field, assocOrComp));
         const externalEntity = (0, combi_1.seq)((0, combi_1.opt)("WRITABLE"), "EXTERNAL", "ENTITY", _1.CDSName, (0, combi_1.opt)(externalName), (0, combi_1.opt)(_1.CDSWithParameters));
         const staticEntity = (0, combi_1.seq)("STATIC", "ENTITY", _1.CDSName, (0, combi_1.opt)(externalName));
-        return (0, combi_1.seq)((0, combi_1.star)(_1.CDSAnnotation), "DEFINE", (0, combi_1.alt)(externalEntity, staticEntity), (0, combi_1.str)("{"), body, (0, combi_1.str)("}"), (0, combi_1.opt)((0, combi_1.seq)("WITH", "FEDERATED", "DATA", (0, combi_1.optPrio)((0, combi_1.alt)((0, combi_1.seq)("PROVIDED", "AT", "RUNTIME"), (0, combi_1.seq)("PROVIDED", "BY", _1.CDSName))))), (0, combi_1.opt)(";"));
+        return (0, combi_1.seq)((0, combi_1.star)(_1.CDSAnnotation), (0, combi_1.opt)("DEFINE"), (0, combi_1.alt)(externalEntity, staticEntity), (0, combi_1.str)("{"), body, (0, combi_1.str)("}"), (0, combi_1.opt)((0, combi_1.seq)("WITH", "FEDERATED", "DATA", (0, combi_1.optPrio)((0, combi_1.alt)((0, combi_1.seq)("PROVIDED", "AT", "RUNTIME"), (0, combi_1.seq)("PROVIDED", "BY", _1.CDSName))))), (0, combi_1.opt)(";"));
     }
 }
 exports.CDSDefineExternalEntity = CDSDefineExternalEntity;
@@ -53704,7 +53793,7 @@ class CDSDefineHierarchy extends combi_1.Expression {
         const depthValue = (0, combi_1.altPrio)(_1.CDSString, _1.CDSInteger, _1.CDSPrefixedName);
         const loadMode = (0, combi_1.altPrio)("BULK", "INCREMENTAL", _1.CDSPrefixedName);
         const hierarchyBody = (0, combi_1.seq)("SOURCE", _1.CDSName, (0, combi_1.opt)(_1.CDSParametersSelect), "CHILD", "TO", "PARENT", "ASSOCIATION", _1.CDSName, (0, combi_1.opt)(directory), (0, combi_1.opt)(datePeriod), (0, combi_1.opt)((0, combi_1.seq)("START", "WHERE", _1.CDSCondition)), (0, combi_1.opt)(siblingsOrder), (0, combi_1.opt)((0, combi_1.seq)("LOAD", loadMode)), (0, combi_1.opt)((0, combi_1.seq)("DEPTH", depthValue)), (0, combi_1.opt)((0, combi_1.seq)("NODETYPE", _1.CDSName)), (0, combi_1.opt)((0, combi_1.seq)("MULTIPLE", "PARENTS", (0, combi_1.altPrio)("NOT ALLOWED", "ALLOWED", (0, combi_1.seq)("LEAVES", (0, combi_1.optPrio)("ONLY"))))), (0, combi_1.opt)((0, combi_1.seq)("ORPHANS", (0, combi_1.altPrio)("IGNORE", "ROOT", "ERROR"))), (0, combi_1.opt)((0, combi_1.seq)("CYCLES", (0, combi_1.altPrio)("BREAKUP", "ERROR"))), (0, combi_1.opt)((0, combi_1.seq)("GENERATE", "SPANTREE")), (0, combi_1.opt)((0, combi_1.seq)("CACHE", (0, combi_1.altPrio)("ON", "OFF", "FORCE"))));
-        return (0, combi_1.seq)((0, combi_1.star)(_1.CDSAnnotation), "DEFINE", "HIERARCHY", _1.CDSName, (0, combi_1.opt)(_1.CDSWithParameters), "AS", "PARENT", "CHILD", "HIERARCHY", "(", hierarchyBody, ")", "{", (0, combi_1.seq)(field, (0, combi_1.star)((0, combi_1.seq)(",", field))), "}", (0, combi_1.opt)(";"));
+        return (0, combi_1.seq)((0, combi_1.star)(_1.CDSAnnotation), (0, combi_1.opt)("DEFINE"), "HIERARCHY", _1.CDSName, (0, combi_1.opt)(_1.CDSWithParameters), "AS", "PARENT", "CHILD", "HIERARCHY", "(", hierarchyBody, ")", "{", (0, combi_1.seq)(field, (0, combi_1.star)((0, combi_1.seq)(",", field))), "}", (0, combi_1.opt)(";"));
     }
 }
 exports.CDSDefineHierarchy = CDSDefineHierarchy;
@@ -53729,7 +53818,7 @@ class CDSDefineProjection extends combi_1.Expression {
     getRunnable() {
         // redefine association _Assoc redirected to [composition child | parent] Entity
         const redefineAssoc = (0, combi_1.seq)("REDEFINE", "ASSOCIATION", _1.CDSName, "REDIRECTED", "TO", (0, combi_1.optPrio)((0, combi_1.altPrio)((0, combi_1.seq)("COMPOSITION", "CHILD"), "PARENT")), _1.CDSName);
-        return (0, combi_1.seq)((0, combi_1.star)(_1.CDSAnnotation), "DEFINE", (0, combi_1.opt)("ROOT"), (0, combi_1.opt)("TRANSIENT"), "VIEW", (0, combi_1.ver)(__1.Release.v755, (0, combi_1.opt)("ENTITY")), _1.CDSName, (0, combi_1.opt)(_1.CDSProviderContract), (0, combi_1.opt)(_1.CDSWithParameters), "AS PROJECTION ON", _1.CDSName, (0, combi_1.opt)(_1.CDSParametersSelect), (0, combi_1.opt)(_1.CDSAs), (0, combi_1.star)((0, combi_1.altPrio)(_1.CDSAssociation, redefineAssoc)), (0, combi_1.str)("{"), (0, combi_1.seq)(_1.CDSElement, (0, combi_1.star)((0, combi_1.seq)(",", _1.CDSElement)), (0, combi_1.opt)(",")), (0, combi_1.str)("}"), (0, combi_1.opt)(_1.CDSWhere), (0, combi_1.opt)(";"));
+        return (0, combi_1.seq)((0, combi_1.star)(_1.CDSAnnotation), (0, combi_1.opt)("DEFINE"), (0, combi_1.opt)("ROOT"), (0, combi_1.opt)("TRANSIENT"), "VIEW", (0, combi_1.ver)(__1.Release.v755, (0, combi_1.opt)("ENTITY")), _1.CDSName, (0, combi_1.opt)(_1.CDSProviderContract), (0, combi_1.opt)(_1.CDSWithParameters), "AS PROJECTION ON", _1.CDSName, (0, combi_1.opt)(_1.CDSParametersSelect), (0, combi_1.opt)(_1.CDSAs), (0, combi_1.star)((0, combi_1.altPrio)(_1.CDSAssociation, redefineAssoc)), (0, combi_1.str)("{"), (0, combi_1.seq)(_1.CDSElement, (0, combi_1.star)((0, combi_1.seq)(",", _1.CDSElement)), (0, combi_1.opt)(",")), (0, combi_1.str)("}"), (0, combi_1.opt)(_1.CDSWhere), (0, combi_1.opt)(";"));
     }
 }
 exports.CDSDefineProjection = CDSDefineProjection;
@@ -53758,7 +53847,7 @@ class CDSDefineTableEntity extends combi_1.Expression {
         const elementList = (0, combi_1.seq)(_1.CDSElement, (0, combi_1.star)((0, combi_1.seq)(",", _1.CDSElement)), (0, combi_1.opt)(","));
         const elements = (0, combi_1.seq)((0, combi_1.str)("{"), (0, combi_1.altPrio)("*", elementList), (0, combi_1.str)("}"));
         const selectBody = (0, combi_1.seq)("AS", "SELECT", "FROM", _1.CDSSource, (0, combi_1.star)(_1.CDSJoin), (0, combi_1.star)((0, combi_1.altPrio)(_1.CDSComposition, _1.CDSAssociation)), (0, combi_1.opt)(elements), (0, combi_1.optPrio)(_1.CDSWhere));
-        return (0, combi_1.seq)((0, combi_1.star)(_1.CDSAnnotation), "DEFINE", (0, combi_1.opt)("ROOT"), "TABLE", "ENTITY", _1.CDSName, (0, combi_1.opt)(_1.CDSWithParameters), (0, combi_1.altPrio)((0, combi_1.seq)((0, combi_1.str)("{"), inlineBody, (0, combi_1.str)("}")), selectBody), (0, combi_1.opt)(";"));
+        return (0, combi_1.seq)((0, combi_1.star)(_1.CDSAnnotation), (0, combi_1.opt)("DEFINE"), (0, combi_1.opt)("ROOT"), "TABLE", "ENTITY", _1.CDSName, (0, combi_1.opt)(_1.CDSWithParameters), (0, combi_1.altPrio)((0, combi_1.seq)((0, combi_1.str)("{"), inlineBody, (0, combi_1.str)("}")), selectBody), (0, combi_1.opt)(";"));
     }
 }
 exports.CDSDefineTableEntity = CDSDefineTableEntity;
@@ -54700,7 +54789,7 @@ class Config {
         }
         return (0, version_1.versionToABAPRelease)(v);
     }
-    getOpenABAP() {
+    isOpenABAP() {
         return this.getRelease() === version_1.Release["open-abap"];
     }
     getVersion() {
@@ -68847,7 +68936,7 @@ class Registry {
     }
     static abaplintVersion() {
         // magic, see build script "version.js"
-        return "2.119.67";
+        return "2.120.1";
     }
     getDDICReferences() {
         return this.ddicReferences;
@@ -73550,6 +73639,270 @@ exports.ClassicExceptionsOverlap = ClassicExceptionsOverlap;
 
 /***/ },
 
+/***/ "../core/build/src/rules/clear_exporting_parameters.js"
+/*!*************************************************************!*\
+  !*** ../core/build/src/rules/clear_exporting_parameters.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.ClearExportingParameters = exports.ClearExportingParametersConf = void 0;
+const issue_1 = __webpack_require__(/*! ../issue */ "../core/build/src/issue.js");
+const _basic_rule_config_1 = __webpack_require__(/*! ./_basic_rule_config */ "../core/build/src/rules/_basic_rule_config.js");
+const _irule_1 = __webpack_require__(/*! ./_irule */ "../core/build/src/rules/_irule.js");
+const syntax_1 = __webpack_require__(/*! ../abap/5_syntax/syntax */ "../core/build/src/abap/5_syntax/syntax.js");
+const _abap_object_1 = __webpack_require__(/*! ../objects/_abap_object */ "../core/build/src/objects/_abap_object.js");
+const _scope_type_1 = __webpack_require__(/*! ../abap/5_syntax/_scope_type */ "../core/build/src/abap/5_syntax/_scope_type.js");
+const objects_1 = __webpack_require__(/*! ../objects */ "../core/build/src/objects/index.js");
+const _reference_1 = __webpack_require__(/*! ../abap/5_syntax/_reference */ "../core/build/src/abap/5_syntax/_reference.js");
+const basic_1 = __webpack_require__(/*! ../abap/types/basic */ "../core/build/src/abap/types/basic/index.js");
+const _statement_1 = __webpack_require__(/*! ../abap/2_statements/statements/_statement */ "../core/build/src/abap/2_statements/statements/_statement.js");
+const edit_helper_1 = __webpack_require__(/*! ../edit_helper */ "../core/build/src/edit_helper.js");
+const Expressions = __importStar(__webpack_require__(/*! ../abap/2_statements/expressions */ "../core/build/src/abap/2_statements/expressions/index.js"));
+class ClearExportingParametersConf extends _basic_rule_config_1.BasicRuleConfig {
+    constructor() {
+        super(...arguments);
+        /** Skip specific parameter names, case insensitive
+         * @uniqueItems true
+         */
+        this.skipNames = [];
+    }
+}
+exports.ClearExportingParametersConf = ClearExportingParametersConf;
+class ClearExportingParameters {
+    constructor() {
+        this.conf = new ClearExportingParametersConf();
+    }
+    getMetadata() {
+        return {
+            key: "clear_exporting_parameters",
+            title: "Clear EXPORTING parameters",
+            shortDescription: `Checks that EXPORTING parameters passed by reference are cleared or assigned before they are read`,
+            extendedInformation: `An EXPORTING parameter passed by reference behaves like a CHANGING parameter: it is not
+initialized when the method is called, so it can still contain a value supplied by the caller. Clear or overwrite it
+before reading it, so a leftover value is not accidentally used.
+
+https://github.com/SAP/styleguides/blob/main/clean-abap/CleanABAP.md#clear-or-overwrite-exporting-reference-parameters
+
+https://help.sap.com/doc/abapdocu_latest_index_htm/latest/en-US/abenref_transf_output_param_guidl.htm
+
+Note: EXPORTING parameters passed by VALUE are always initialized and are therefore not reported.
+Reading and writing the parameter in the same statement (e.g. "ev_result = ev_result + 1") is reported,
+as the source is evaluated before the parameter is assigned.
+Objects containing parser or syntax errors are not reported.
+
+Only class and interface method implementations are checked, function module parameters are currently not supported.`,
+            tags: [_irule_1.RuleTag.Styleguide],
+            badExample: `CLASS lcl DEFINITION.
+  PUBLIC SECTION.
+    METHODS foo EXPORTING ev_result TYPE i.
+ENDCLASS.
+CLASS lcl IMPLEMENTATION.
+  METHOD foo.
+    ev_result = ev_result + 1.
+  ENDMETHOD.
+ENDCLASS.`,
+            goodExample: `CLASS lcl DEFINITION.
+  PUBLIC SECTION.
+    METHODS foo EXPORTING ev_result TYPE i.
+ENDCLASS.
+CLASS lcl IMPLEMENTATION.
+  METHOD foo.
+    CLEAR ev_result.
+    ev_result = ev_result + 1.
+  ENDMETHOD.
+ENDCLASS.`,
+        };
+    }
+    getConfig() {
+        return this.conf;
+    }
+    setConfig(conf) {
+        this.conf = conf;
+        if (this.conf.skipNames === undefined) {
+            this.conf.skipNames = [];
+        }
+    }
+    initialize(reg) {
+        this.reg = reg;
+        return this;
+    }
+    run(obj) {
+        if (!(obj instanceof _abap_object_1.ABAPObject) || obj instanceof objects_1.Interface) {
+            return [];
+        }
+        for (const file of obj.getABAPFiles()) {
+            for (const statement of file.getStatements()) {
+                if (statement.get() instanceof _statement_1.Unknown) {
+                    return []; // contains parser errors
+                }
+            }
+        }
+        const syntax = new syntax_1.SyntaxLogic(this.reg, obj).run();
+        if (syntax.issues.length > 0) {
+            return []; // contains syntax errors
+        }
+        const issues = [];
+        this.traverse(syntax.spaghetti.getTop(), obj, issues);
+        return issues;
+    }
+    traverse(node, obj, issues) {
+        if (node.getIdentifier().stype === _scope_type_1.ScopeType.Method) {
+            this.checkMethod(node, obj, issues);
+        }
+        for (const child of node.getChildren()) {
+            this.traverse(child, obj, issues);
+        }
+    }
+    checkMethod(node, obj, issues) {
+        const parameters = this.findExportingByReference(node, obj);
+        if (parameters.length === 0) {
+            return;
+        }
+        const references = this.collectReferences(node);
+        for (const parameter of parameters) {
+            const issue = this.checkParameter(parameter, references, obj);
+            if (issue !== undefined) {
+                issues.push(issue);
+            }
+        }
+    }
+    findExportingByReference(node, obj) {
+        var _a;
+        const ret = [];
+        const vars = node.getData().vars;
+        for (const name in vars) {
+            const parameter = vars[name];
+            const meta = parameter.getMeta();
+            if (meta.includes("exporting" /* IdentifierMeta.MethodExporting */) === false
+                || meta.includes("pass_by_value" /* IdentifierMeta.PassByValue */) === true) {
+                continue;
+            }
+            else if ((_a = this.conf.skipNames) === null || _a === void 0 ? void 0 : _a.some(s => s.toUpperCase() === name.toUpperCase())) {
+                continue;
+            }
+            const type = parameter.getType();
+            if (type instanceof basic_1.VoidType || type instanceof basic_1.UnknownType) {
+                continue; // e.g. RAP magic parameters, or unresolved types
+            }
+            else if (this.isPassByValue(parameter, obj) === true) {
+                continue; // VALUE(..) parameters are always initialized
+            }
+            ret.push(parameter);
+        }
+        return ret;
+    }
+    isPassByValue(parameter, obj) {
+        var _a;
+        // the PassByValue meta is not set for EXPORTING parameters, so determine it from the definition
+        const file = obj.getABAPFileByName(parameter.getFilename());
+        if (file === undefined) {
+            return false;
+        }
+        const statement = edit_helper_1.EditHelper.findStatement(parameter.getToken(), file);
+        if (statement === undefined) {
+            return false;
+        }
+        for (const param of statement.findAllExpressions(Expressions.MethodParam)) {
+            const nameToken = (_a = param.findFirstExpression(Expressions.MethodParamName)) === null || _a === void 0 ? void 0 : _a.getFirstToken();
+            if (nameToken !== undefined && nameToken.getStart().equals(parameter.getStart())) {
+                return param.getFirstToken().getStr().toUpperCase() === "VALUE";
+            }
+        }
+        return false;
+    }
+    collectReferences(node) {
+        // methods do not nest, so all descendant scopes (FOR, LET, ...) belong to this method
+        const ret = [...node.getData().references];
+        for (const child of node.getChildren()) {
+            ret.push(...this.collectReferences(child));
+        }
+        return ret;
+    }
+    checkParameter(parameter, references, obj) {
+        let firstRead = undefined;
+        let earliestWrite = undefined;
+        for (const reference of references) {
+            if (reference.resolved === undefined || parameter.equals(reference.resolved) === false) {
+                continue;
+            }
+            const pos = reference.position.getStart();
+            if (reference.referenceType === _reference_1.ReferenceType.DataReadReference) {
+                if (firstRead === undefined || pos.isBefore(firstRead.position.getStart())) {
+                    firstRead = reference;
+                }
+            }
+            else if (reference.referenceType === _reference_1.ReferenceType.DataWriteReference) {
+                if (earliestWrite === undefined || pos.isBefore(earliestWrite)) {
+                    earliestWrite = pos;
+                }
+            }
+        }
+        if (firstRead === undefined) {
+            return undefined; // never read, no hazard
+        }
+        // the parameter is safe only if it is written in a statement strictly before the first read
+        const readStatement = this.statementStart(firstRead.position.getStart(), obj);
+        const writeStatement = earliestWrite === undefined ? undefined : this.statementStart(earliestWrite, obj);
+        if (writeStatement !== undefined && readStatement !== undefined && writeStatement.isBefore(readStatement)) {
+            return undefined;
+        }
+        const message = `EXPORTING parameter "${parameter.getName().toLowerCase()}" read before it is cleared or assigned`;
+        return issue_1.Issue.atIdentifier(firstRead.position, message, this.getMetadata().key, this.conf.severity);
+    }
+    statementStart(pos, obj) {
+        for (const file of obj.getABAPFiles()) {
+            for (const statement of file.getStatements()) {
+                const start = statement.getStart();
+                const end = statement.getEnd();
+                if (pos.equals(start) || (pos.isAfter(start) && pos.isBefore(end))) {
+                    return start;
+                }
+            }
+        }
+        return undefined;
+    }
+}
+exports.ClearExportingParameters = ClearExportingParameters;
+//# sourceMappingURL=clear_exporting_parameters.js.map
+
+/***/ },
+
 /***/ "../core/build/src/rules/cloud_types.js"
 /*!**********************************************!*\
   !*** ../core/build/src/rules/cloud_types.js ***!
@@ -75465,7 +75818,7 @@ Make sure to test the downported code, it might not always be completely correct
     initialize(reg) {
         this.lowReg = reg;
         const config = this.lowReg.getConfig();
-        if (config.getRelease() === version_1.Release.v702 || config.getOpenABAP()) {
+        if (config.getRelease() === version_1.Release.v702 || config.isOpenABAP()) {
             this.initHighReg();
         }
         return this;
@@ -75498,7 +75851,7 @@ Make sure to test the downported code, it might not always be completely correct
         const ret = [];
         this.counter = 1;
         const config = this.lowReg.getConfig();
-        if (config.getRelease() !== version_1.Release.v702 && !config.getOpenABAP()) {
+        if (config.getRelease() !== version_1.Release.v702 && !config.isOpenABAP()) {
             return ret;
         }
         else if (!(lowObj instanceof _abap_object_1.ABAPObject)) {
@@ -76370,7 +76723,7 @@ ${indentation}`);
     outlineCatchSimple(node, lowFile) {
         // outlines "CATCH cx_bcs INTO DATA(lx_bcs_excep).", note that this does not need to look at types
         var _a, _b;
-        if (this.lowReg.getConfig().getOpenABAP()) {
+        if (this.lowReg.getConfig().isOpenABAP()) {
             return undefined;
         }
         if (!(node.get() instanceof Statements.Catch)) {
@@ -76419,7 +76772,7 @@ ${indentation}CATCH ${className} INTO ${targetName}.`;
         if (!(node.get() instanceof Statements.Move)) {
             return undefined;
         }
-        if (this.lowReg.getConfig().getOpenABAP()) {
+        if (this.lowReg.getConfig().isOpenABAP()) {
             return undefined;
         }
         const target = node.findFirstExpression(Expressions.Target);
@@ -76475,7 +76828,7 @@ ${indentation}CATCH ${className} INTO ${targetName}.`;
         return issue_1.Issue.atToken(lowFile, node.getFirstToken(), "Outline DATA", this.getMetadata().key, this.conf.severity, fix);
     }
     partiallyImplemented(node, lowFile) {
-        if (this.lowReg.getConfig().getOpenABAP()) {
+        if (this.lowReg.getConfig().isOpenABAP()) {
             return undefined;
         }
         if (node.get() instanceof Statements.InterfaceDef) {
@@ -77151,7 +77504,7 @@ ${indentation}    output = ${uniqueName}.\n`;
         if (!(node.get() instanceof Statements.Loop)) {
             return undefined;
         }
-        if (this.lowReg.getConfig().getOpenABAP()) {
+        if (this.lowReg.getConfig().isOpenABAP()) {
             return undefined;
         }
         const source = (_a = node.findDirectExpression(Expressions.LoopSource)) === null || _a === void 0 ? void 0 : _a.findDirectExpression(Expressions.SimpleSource2);
@@ -77827,7 +78180,7 @@ ${indentation}    output = ${uniqueName}.\n`;
     outlineData(node, lowFile, highSyntax) {
         // hmm, no guard here, as DATA(SDF) is valid in 702
         var _a, _b;
-        if (this.lowReg.getConfig().getOpenABAP()) {
+        if (this.lowReg.getConfig().isOpenABAP()) {
             return undefined;
         }
         for (const i of node.findAllExpressionsRecursive(Expressions.InlineData)) {
@@ -78046,7 +78399,7 @@ ${indentation}    output = ${uniqueName}.\n`;
         return false;
     }
     replaceXsdBool(node, lowFile, highSyntax) {
-        if (this.lowReg.getConfig().getOpenABAP()) {
+        if (this.lowReg.getConfig().isOpenABAP()) {
             return undefined;
         }
         const spag = highSyntax.spaghetti.lookupPosition(node.getFirstToken().getStart(), lowFile.getFilename());
@@ -78175,7 +78528,7 @@ ${indentation}    output = ${uniqueName}.\n`;
     }
     replaceLineFunctions(node, lowFile, highSyntax, highFile) {
         var _a, _b;
-        if (this.lowReg.getConfig().getOpenABAP()) {
+        if (this.lowReg.getConfig().isOpenABAP()) {
             return undefined;
         }
         const spag = highSyntax.spaghetti.lookupPosition(node.getFirstToken().getStart(), lowFile.getFilename());
@@ -82349,6 +82702,7 @@ __exportStar(__webpack_require__(/*! ./check_subrc */ "../core/build/src/rules/c
 __exportStar(__webpack_require__(/*! ./check_syntax */ "../core/build/src/rules/check_syntax.js"), exports);
 __exportStar(__webpack_require__(/*! ./check_text_elements */ "../core/build/src/rules/check_text_elements.js"), exports);
 __exportStar(__webpack_require__(/*! ./check_transformation_exists */ "../core/build/src/rules/check_transformation_exists.js"), exports);
+__exportStar(__webpack_require__(/*! ./clear_exporting_parameters */ "../core/build/src/rules/clear_exporting_parameters.js"), exports);
 __exportStar(__webpack_require__(/*! ./class_attribute_names */ "../core/build/src/rules/class_attribute_names.js"), exports);
 __exportStar(__webpack_require__(/*! ./classic_exceptions_overlap */ "../core/build/src/rules/classic_exceptions_overlap.js"), exports);
 __exportStar(__webpack_require__(/*! ./cloud_types */ "../core/build/src/rules/cloud_types.js"), exports);
@@ -82454,6 +82808,7 @@ __exportStar(__webpack_require__(/*! ./preferred_compare_operator */ "../core/bu
 __exportStar(__webpack_require__(/*! ./prefix_is_current_class */ "../core/build/src/rules/prefix_is_current_class.js"), exports);
 __exportStar(__webpack_require__(/*! ./reduce_procedural_code */ "../core/build/src/rules/reduce_procedural_code.js"), exports);
 __exportStar(__webpack_require__(/*! ./reduce_string_templates */ "../core/build/src/rules/reduce_string_templates.js"), exports);
+__exportStar(__webpack_require__(/*! ./redundant_conversion */ "../core/build/src/rules/redundant_conversion.js"), exports);
 __exportStar(__webpack_require__(/*! ./release_idoc */ "../core/build/src/rules/release_idoc.js"), exports);
 __exportStar(__webpack_require__(/*! ./remove_descriptions */ "../core/build/src/rules/remove_descriptions.js"), exports);
 __exportStar(__webpack_require__(/*! ./rfc_error_handling */ "../core/build/src/rules/rfc_error_handling.js"), exports);
@@ -82659,7 +83014,7 @@ class InlineDataOldVersions extends _abap_rule_1.ABAPRule {
     runParsed(file) {
         const issues = [];
         if ((0, version_1.releaseAtLeast)(this.reg.getConfig().getRelease(), version_1.Release.v740sp02)
-            || this.reg.getConfig().getOpenABAP()
+            || this.reg.getConfig().isOpenABAP()
             || this.reg.getConfig().getLanguageVersion() === version_1.LanguageVersion.Cloud) {
             return [];
         }
@@ -86781,7 +87136,7 @@ Not considered optional branches:
         const release = this.reg.getConfig().getRelease();
         if (release === version_1.Release.v700
             || release === version_1.Release.v702
-            || this.reg.getConfig().getOpenABAP()) {
+            || this.reg.getConfig().isOpenABAP()) {
             return [];
         }
         const struc = file.getStructure();
@@ -90932,6 +91287,269 @@ exports.ReduceStringTemplates = ReduceStringTemplates;
 
 /***/ },
 
+/***/ "../core/build/src/rules/redundant_conversion.js"
+/*!*******************************************************!*\
+  !*** ../core/build/src/rules/redundant_conversion.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.RedundantConversion = exports.RedundantConversionConf = void 0;
+const Expressions = __importStar(__webpack_require__(/*! ../abap/2_statements/expressions */ "../core/build/src/abap/2_statements/expressions/index.js"));
+const Statements = __importStar(__webpack_require__(/*! ../abap/2_statements/statements */ "../core/build/src/abap/2_statements/statements/index.js"));
+const _reference_1 = __webpack_require__(/*! ../abap/5_syntax/_reference */ "../core/build/src/abap/5_syntax/_reference.js");
+const constant_1 = __webpack_require__(/*! ../abap/5_syntax/expressions/constant */ "../core/build/src/abap/5_syntax/expressions/constant.js");
+const syntax_1 = __webpack_require__(/*! ../abap/5_syntax/syntax */ "../core/build/src/abap/5_syntax/syntax.js");
+const nodes_1 = __webpack_require__(/*! ../abap/nodes */ "../core/build/src/abap/nodes/index.js");
+const _typed_identifier_1 = __webpack_require__(/*! ../abap/types/_typed_identifier */ "../core/build/src/abap/types/_typed_identifier.js");
+const basic_1 = __webpack_require__(/*! ../abap/types/basic */ "../core/build/src/abap/types/basic/index.js");
+const issue_1 = __webpack_require__(/*! ../issue */ "../core/build/src/issue.js");
+const _abap_object_1 = __webpack_require__(/*! ../objects/_abap_object */ "../core/build/src/objects/_abap_object.js");
+const _basic_rule_config_1 = __webpack_require__(/*! ./_basic_rule_config */ "../core/build/src/rules/_basic_rule_config.js");
+const _irule_1 = __webpack_require__(/*! ./_irule */ "../core/build/src/rules/_irule.js");
+const edit_helper_1 = __webpack_require__(/*! ../edit_helper */ "../core/build/src/edit_helper.js");
+const version_1 = __webpack_require__(/*! ../version */ "../core/build/src/version.js");
+class RedundantConversionConf extends _basic_rule_config_1.BasicRuleConfig {
+}
+exports.RedundantConversionConf = RedundantConversionConf;
+class RedundantConversion {
+    constructor() {
+        this.conf = new RedundantConversionConf();
+    }
+    getMetadata() {
+        return {
+            key: "redundant_conversion",
+            title: "Redundant Conversion",
+            shortDescription: `Find redundant CONV expressions`,
+            extendedInformation: `Reports CONV expressions whose operand already has the conversion's target type.`,
+            tags: [_irule_1.RuleTag.Quickfix],
+            badExample: `DATA text TYPE string.
+text = CONV string( text ).`,
+            goodExample: `DATA text TYPE string.
+text = text.`,
+        };
+    }
+    getConfig() {
+        return this.conf;
+    }
+    setConfig(conf) {
+        this.conf = conf;
+    }
+    initialize(reg) {
+        this.reg = reg;
+        return this;
+    }
+    run(obj) {
+        var _a, _b;
+        if (!(0, version_1.releaseAtLeast)(this.reg.getConfig().getRelease(), version_1.Release.v740sp02)
+            && !this.reg.getConfig().isOpenABAP()) {
+            return [];
+        }
+        if (!(obj instanceof _abap_object_1.ABAPObject)) {
+            return [];
+        }
+        const syntax = new syntax_1.SyntaxLogic(this.reg, obj).run();
+        if (syntax.issues.length > 0) {
+            return [];
+        }
+        const issues = [];
+        for (const file of obj.getABAPFiles()) {
+            const structure = file.getStructure();
+            if (structure === undefined) {
+                continue;
+            }
+            for (const source of structure.findAllExpressions(Expressions.Source)) {
+                if (source.getFirstToken().getStr().toUpperCase() !== "CONV") {
+                    continue;
+                }
+                const typeExpression = source.findDirectExpression(Expressions.TypeNameOrInfer);
+                const bodySource = (_a = source.findDirectExpression(Expressions.ConvBody)) === null || _a === void 0 ? void 0 : _a.findDirectExpression(Expressions.Source);
+                if (typeExpression === undefined
+                    || bodySource === undefined) {
+                    continue;
+                }
+                const scope = syntax.spaghetti.lookupPosition(source.getFirstToken().getStart(), file.getFilename());
+                if (scope === undefined) {
+                    continue;
+                }
+                const target = (_b = scope.getData().references.find(reference => reference.referenceType === _reference_1.ReferenceType.InferredType
+                    && reference.position.getStart().equals(typeExpression.getFirstToken().getStart()))) === null || _b === void 0 ? void 0 : _b.resolved;
+                const operandType = this.sourceType(bodySource, scope);
+                if (!(target instanceof _typed_identifier_1.TypedIdentifier)) {
+                    continue;
+                }
+                const operandHasSameType = operandType !== undefined && this.sameType(target.getType(), operandType);
+                const arithmeticHasSameContext = this.arithmeticUsesSameTargetType(source, bodySource, file, scope, target.getType());
+                if (operandHasSameType === false && arithmeticHasSameContext === false) {
+                    continue;
+                }
+                const closingParen = source.getDirectTokens().find(token => token.getStr() === ")" && token.getStart().isAfter(bodySource.getLastToken().getStart()));
+                if (closingParen === undefined) {
+                    continue;
+                }
+                const preserveParentheses = bodySource.findDirectExpression(Expressions.ArithOperator) !== undefined
+                    || bodySource.findDirectTokenByText("&&") !== undefined;
+                // Apply the later edit first so the original positions remain valid when
+                // both parts of the CONV wrapper are changed on the same line.
+                const fix = edit_helper_1.EditHelper.merge(edit_helper_1.EditHelper.replaceRange(file, bodySource.getLastToken().getEnd(), closingParen.getEnd(), preserveParentheses ? ")" : ""), edit_helper_1.EditHelper.replaceRange(file, source.getFirstToken().getStart(), bodySource.getFirstToken().getStart(), preserveParentheses ? "(" : ""));
+                issues.push(issue_1.Issue.atRange(file, source.getFirstToken().getStart(), closingParen.getEnd(), "Redundant CONV expression", this.getMetadata().key, this.conf.severity, fix));
+            }
+        }
+        return issues;
+    }
+    arithmeticUsesSameTargetType(conversion, bodySource, file, scope, conversionType) {
+        var _a, _b;
+        if (bodySource.findDirectExpression(Expressions.ArithOperator) === undefined) {
+            return false;
+        }
+        const statement = file.getStatements().find(candidate => candidate.includesToken(conversion.getFirstToken()));
+        if (!((statement === null || statement === void 0 ? void 0 : statement.get()) instanceof Statements.Move)) {
+            return false;
+        }
+        const targetToken = (_a = statement.findDirectExpression(Expressions.Target)) === null || _a === void 0 ? void 0 : _a.getFirstToken();
+        if (targetToken === undefined) {
+            return false;
+        }
+        const assignmentTarget = (_b = scope.getData().references.find(reference => reference.referenceType === _reference_1.ReferenceType.DataWriteReference
+            && reference.position.getStart().equals(targetToken.getStart()))) === null || _b === void 0 ? void 0 : _b.resolved;
+        return assignmentTarget instanceof _typed_identifier_1.TypedIdentifier
+            && this.sameType(conversionType, assignmentTarget.getType());
+    }
+    sourceType(source, scope) {
+        let context;
+        for (const child of source.getChildren()) {
+            if (!(child instanceof nodes_1.ExpressionNode)) {
+                continue;
+            }
+            else if (child.get() instanceof Expressions.FieldChain) {
+                context = this.fieldChainType(child, scope);
+                if (context === undefined) {
+                    return undefined;
+                }
+            }
+            else if (child.get() instanceof Expressions.Constant) {
+                context = this.infer(context, constant_1.Constant.runSyntax(child));
+            }
+            else if (child.get() instanceof Expressions.Source) {
+                context = this.infer(context, this.sourceType(child, scope));
+            }
+            else if (child.get() instanceof Expressions.ArithOperator) {
+                if (child.concatTokens() === "**") {
+                    context = new basic_1.FloatType();
+                }
+            }
+            else {
+                return undefined;
+            }
+        }
+        if (source.findDirectTokenByText("&&")) {
+            return basic_1.StringType.get();
+        }
+        return context;
+    }
+    fieldChainType(fieldChain, scope) {
+        var _a;
+        const operand = (_a = scope.getData().references.find(reference => reference.referenceType === _reference_1.ReferenceType.DataReadReference
+            && reference.position.getStart().equals(fieldChain.getFirstToken().getStart()))) === null || _a === void 0 ? void 0 : _a.resolved;
+        if (!(operand instanceof _typed_identifier_1.TypedIdentifier)) {
+            return undefined;
+        }
+        let type = operand.getType();
+        for (const child of fieldChain.getChildren()) {
+            if (!(child instanceof nodes_1.ExpressionNode)
+                || child.get() instanceof Expressions.SourceField) {
+                continue;
+            }
+            else if (child.get() instanceof Expressions.ComponentName) {
+                if (type instanceof basic_1.TableType) {
+                    type = type.getRowType();
+                }
+                if (!(type instanceof basic_1.StructureType)) {
+                    return undefined;
+                }
+                type = type.getComponentByName(child.concatTokens());
+            }
+            else {
+                // Offsets, table expressions, attributes, and dereferences need more
+                // context than the initial data reference provides.
+                return undefined;
+            }
+        }
+        return type;
+    }
+    infer(context, found) {
+        if (context instanceof basic_1.FloatType && found instanceof basic_1.IntegerType) {
+            return context;
+        }
+        else if (context instanceof basic_1.IntegerType && found instanceof basic_1.HexType) {
+            return context;
+        }
+        else if ((context instanceof basic_1.IntegerType || context instanceof basic_1.FloatType) && (found === null || found === void 0 ? void 0 : found.isGeneric())) {
+            return context;
+        }
+        return found;
+    }
+    sameType(first, second) {
+        var _a, _b;
+        if (first instanceof basic_1.UnknownType
+            || second instanceof basic_1.UnknownType
+            || first instanceof basic_1.VoidType
+            || second instanceof basic_1.VoidType
+            || first.isGeneric()
+            || second.isGeneric()) {
+            return false;
+        }
+        if (first.constructor !== second.constructor) {
+            return false;
+        }
+        if (first instanceof basic_1.StructureType || first instanceof basic_1.TableType) {
+            const firstName = (_a = first.getQualifiedName()) === null || _a === void 0 ? void 0 : _a.toUpperCase();
+            const secondName = (_b = second.getQualifiedName()) === null || _b === void 0 ? void 0 : _b.toUpperCase();
+            return firstName !== undefined && firstName === secondName;
+        }
+        return first.toABAP() === second.toABAP();
+    }
+}
+exports.RedundantConversion = RedundantConversion;
+//# sourceMappingURL=redundant_conversion.js.map
+
+/***/ },
+
 /***/ "../core/build/src/rules/release_idoc.js"
 /*!***********************************************!*\
   !*** ../core/build/src/rules/release_idoc.js ***!
@@ -92884,7 +93502,7 @@ class SQLEscapeHostVariables extends _abap_rule_1.ABAPRule {
         }
         if (!(0, version_1.releaseAtLeast)(this.reg.getConfig().getRelease(), version_1.Release.v740sp02)
             && this.reg.getConfig().getLanguageVersion() !== version_1.LanguageVersion.Cloud
-            && !this.reg.getConfig().getOpenABAP()) {
+            && !this.reg.getConfig().isOpenABAP()) {
             return [];
         }
         for (const s of file.getStatements()) {
@@ -96922,7 +97540,7 @@ ENDIF.`,
         }
         const release = this.reg.getConfig().getRelease();
         const langVers = this.reg.getConfig().getLanguageVersion();
-        if (this.reg.getConfig().getOpenABAP()) {
+        if (this.reg.getConfig().isOpenABAP()) {
             return [];
         }
         else if (!(0, version_1.releaseAtLeast)(release, version_1.Release.v740sp02) && langVers !== version_1.LanguageVersion.Cloud) {

package/package.json

@@ -1,6 +1,6 @@
 {
     "name": "@abaplint/cli",
-    "version": "2.119.67",
+    "version": "2.120.1",
     "description": "abaplint - Command Line Interface",
     "funding": "https://github.com/sponsors/larshp",
     "bin": {
@@ -39,7 +39,7 @@
     },
     "homepage": "https://abaplint.org",
     "devDependencies": {
-        "@abaplint/core": "^2.119.67",
+        "@abaplint/core": "^2.120.1",
         "@types/chai": "^4.3.20",
         "@types/minimist": "^1.2.5",
         "@types/mocha": "^10.0.10",