@abaplint/transpiler-cli 2.14.5 → 2.14.7

package/build/bundle.js

@@ -4135,7 +4135,7 @@ class ExpandMacros {
     listMacroNames() {
         return this.macros.listMacroNames();
     }
-    find(statements, file, clear = true) {
+    find(statements, file, clear = true, includePath = new Set()) {
         var _a, _b, _c;
         let nameToken = undefined;
         let start = undefined;
@@ -4156,14 +4156,16 @@ class ExpandMacros {
             else if (type instanceof Statements.Include) {
                 const includeName = (_b = statement.findDirectExpression(Expressions.IncludeName)) === null || _b === void 0 ? void 0 : _b.concatTokens();
                 // todo, this does not take function module includes into account
-                // todo, workaround for cyclic includes?
                 const prog = (_c = this.reg) === null || _c === void 0 ? void 0 : _c.getObject("PROG", includeName);
                 if (prog) {
                     prog.parse(this.release, this.globalMacros, this.reg, this.languageVersion);
                     const includeMainFile = prog.getMainABAPFile();
-                    if (includeMainFile) {
+                    // an include that is already on the current include path is not walked again
+                    if (includeMainFile && includePath.has(includeMainFile.getFilename()) === false) {
+                        includePath.add(includeMainFile.getFilename());
                         // slow, this copies everything,
-                        this.find([...includeMainFile.getStatements()], includeMainFile, false);
+                        this.find([...includeMainFile.getStatements()], includeMainFile, false, includePath);
+                        includePath.delete(includeMainFile.getFilename());
                     }
                 }
             }
@@ -14418,11 +14420,11 @@ class Convert {
         const dat = (0, combi_1.seq)("DATE", expressions_1.Source);
         const tim = (0, combi_1.seq)("TIME", expressions_1.Source);
         const stamp = (0, combi_1.seq)("INTO TIME STAMP", expressions_1.Target);
-        const intoutc = (0, combi_1.ver)(version_1.Release.v754, (0, combi_1.seq)("INTO UTCLONG", expressions_1.Target));
+        const intoutc = (0, combi_1.ver)(version_1.Release.v754, (0, combi_1.seq)("INTO UTCLONG", expressions_1.Target), { also: combi_1.AlsoIn.OpenABAP });
         const invert = (0, combi_1.seq)("INTO INVERTED-DATE", expressions_1.Target);
         const date = (0, combi_1.seq)((0, combi_1.per)(dat, tim), (0, combi_1.per)(daylightSource, stamp, zone, invert, intoutc));
         const inv = (0, combi_1.seq)("INVERTED-DATE", expressions_1.Source, "INTO DATE", expressions_1.Target);
-        const utclong = (0, combi_1.ver)(version_1.Release.v754, (0, combi_1.seq)("UTCLONG", expressions_1.Source, (0, combi_1.per)(zone, into, daylightSource)), { also: combi_1.AlsoIn.OpenABAP });
+        const utclong = (0, combi_1.ver)(version_1.Release.v754, (0, combi_1.seq)("UTCLONG", expressions_1.Source, (0, combi_1.per)(zone, into, daylightTarget)), { also: combi_1.AlsoIn.OpenABAP });
         return (0, combi_1.seq)("CONVERT", (0, combi_1.alt)(time, date, inv, utclong));
     }
 }
@@ -26338,28 +26340,28 @@ BuiltIn.methods = {
     "ABS": {
         counter: BuiltIn.counter++,
         mandatory: {
-            "val": new basic_1.FloatType(),
+            "val": new basic_1.NumericGenericType(),
         },
         return: basic_1.IntegerType.get(),
     },
     "ACOS": {
         counter: BuiltIn.counter++,
         mandatory: {
-            "val": new basic_1.FloatType(),
+            "val": new basic_1.NumericGenericType(),
         },
         return: new basic_1.FloatType(),
     },
     "ASIN": {
         counter: BuiltIn.counter++,
         mandatory: {
-            "val": new basic_1.FloatType(),
+            "val": new basic_1.NumericGenericType(),
         },
         return: new basic_1.FloatType(),
     },
     "ATAN": {
         counter: BuiltIn.counter++,
         mandatory: {
-            "val": new basic_1.FloatType(),
+            "val": new basic_1.NumericGenericType(),
         },
         return: new basic_1.FloatType(),
     },
@@ -26393,7 +26395,7 @@ BuiltIn.methods = {
     "CEIL": {
         counter: BuiltIn.counter++,
         mandatory: {
-            "val": new basic_1.FloatType(),
+            "val": new basic_1.NumericGenericType(),
         },
         return: basic_1.IntegerType.get(),
     },
@@ -26530,14 +26532,14 @@ BuiltIn.methods = {
     "COS": {
         counter: BuiltIn.counter++,
         mandatory: {
-            "val": new basic_1.FloatType(),
+            "val": new basic_1.NumericGenericType(),
         },
         return: new basic_1.FloatType(),
     },
     "COSH": {
         counter: BuiltIn.counter++,
         mandatory: {
-            "val": new basic_1.FloatType(),
+            "val": new basic_1.NumericGenericType(),
         },
         return: new basic_1.FloatType(),
     },
@@ -26617,7 +26619,7 @@ BuiltIn.methods = {
     "EXP": {
         counter: BuiltIn.counter++,
         mandatory: {
-            "val": new basic_1.FloatType(),
+            "val": new basic_1.NumericGenericType(),
         },
         return: new basic_1.FloatType(),
     },
@@ -26686,14 +26688,14 @@ BuiltIn.methods = {
     "FLOOR": {
         counter: BuiltIn.counter++,
         mandatory: {
-            "val": new basic_1.FloatType(),
+            "val": new basic_1.NumericGenericType(),
         },
         return: basic_1.IntegerType.get(),
     },
     "FRAC": {
         counter: BuiltIn.counter++,
         mandatory: {
-            "val": new basic_1.FloatType(),
+            "val": new basic_1.NumericGenericType(),
         },
         return: basic_1.IntegerType.get(),
     },
@@ -26725,8 +26727,8 @@ BuiltIn.methods = {
     "IPOW": {
         counter: BuiltIn.counter++,
         mandatory: {
-            "base": new basic_1.FloatType(),
-            "exp": new basic_1.FloatType(),
+            "base": new basic_1.NumericGenericType(),
+            "exp": new basic_1.NumericGenericType(),
         },
         return: basic_1.IntegerType.get(),
         release: version_1.Release.v740sp02,
@@ -26758,14 +26760,14 @@ BuiltIn.methods = {
     "LOG": {
         counter: BuiltIn.counter++,
         mandatory: {
-            "val": new basic_1.FloatType(),
+            "val": new basic_1.NumericGenericType(),
         },
         return: new basic_1.FloatType(),
     },
     "LOG10": {
         counter: BuiltIn.counter++,
         mandatory: {
-            "val": new basic_1.FloatType(),
+            "val": new basic_1.NumericGenericType(),
         },
         return: new basic_1.FloatType(),
     },
@@ -26871,7 +26873,7 @@ BuiltIn.methods = {
     "RESCALE": {
         counter: BuiltIn.counter++,
         mandatory: {
-            "val": new basic_1.FloatType(),
+            "val": new basic_1.NumericGenericType(),
         },
         optional: {
             "dec": basic_1.IntegerType.get(),
@@ -26892,7 +26894,7 @@ BuiltIn.methods = {
     "ROUND": {
         counter: BuiltIn.counter++,
         mandatory: {
-            "val": new basic_1.FloatType(),
+            "val": new basic_1.NumericGenericType(),
         },
         optional: {
             "dec": basic_1.IntegerType.get(),
@@ -26944,28 +26946,28 @@ BuiltIn.methods = {
     "SIGN": {
         counter: BuiltIn.counter++,
         mandatory: {
-            "val": new basic_1.FloatType(),
+            "val": new basic_1.NumericGenericType(),
         },
         return: basic_1.IntegerType.get(),
     },
     "SIN": {
         counter: BuiltIn.counter++,
         mandatory: {
-            "val": new basic_1.FloatType(),
+            "val": new basic_1.NumericGenericType(),
         },
         return: new basic_1.FloatType(),
     },
     "SINH": {
         counter: BuiltIn.counter++,
         mandatory: {
-            "val": new basic_1.FloatType(),
+            "val": new basic_1.NumericGenericType(),
         },
         return: new basic_1.FloatType(),
     },
     "SQRT": {
         counter: BuiltIn.counter++,
         mandatory: {
-            "val": new basic_1.FloatType(),
+            "val": new basic_1.NumericGenericType(),
         },
         return: new basic_1.FloatType(),
     },
@@ -27055,14 +27057,14 @@ BuiltIn.methods = {
     "TAN": {
         counter: BuiltIn.counter++,
         mandatory: {
-            "val": new basic_1.FloatType(),
+            "val": new basic_1.NumericGenericType(),
         },
         return: new basic_1.FloatType(),
     },
     "TANH": {
         counter: BuiltIn.counter++,
         mandatory: {
-            "val": new basic_1.FloatType(),
+            "val": new basic_1.NumericGenericType(),
         },
         return: new basic_1.FloatType(),
     },
@@ -27106,7 +27108,7 @@ BuiltIn.methods = {
     "TRUNC": {
         counter: BuiltIn.counter++,
         mandatory: {
-            "val": new basic_1.FloatType(),
+            "val": new basic_1.NumericGenericType(),
         },
         return: basic_1.IntegerType.get(),
     },
@@ -27117,9 +27119,9 @@ BuiltIn.methods = {
         },
         optional: {
             "days": basic_1.IntegerType.get(),
-            "hour": basic_1.IntegerType.get(),
-            "minutes": basic_1.IntegerType.get(),
-            "seconds": new basic_1.FloatType(),
+            "hours": basic_1.IntegerType.get(),
+            "minutes": new basic_1.Integer8Type(),
+            "seconds": new basic_1.DecFloat34Type(),
         },
         return: new basic_1.UTCLongType(),
         release: version_1.Release.v754,
@@ -27135,7 +27137,7 @@ BuiltIn.methods = {
             "high": new basic_1.UTCLongType(),
             "low": new basic_1.UTCLongType(),
         },
-        return: new basic_1.FloatType(),
+        return: new basic_1.DecFloat34Type(),
         release: version_1.Release.v754,
     },
     "XSDBOOL": {
@@ -28487,7 +28489,12 @@ class Procedural {
         this.scope = scope;
         this.reg = reg;
     }
-    addAllFormDefinitions(file, obj) {
+    addAllFormDefinitions(file, obj, visited = new Set()) {
+        // an include reached again through a cycle of INCLUDEs is not walked twice
+        if (visited.has(file.getFilename())) {
+            return;
+        }
+        visited.add(file.getFilename());
         const structure = file.getStructure();
         if (structure) {
             const input = {
@@ -28508,12 +28515,12 @@ class Procedural {
         for (const node of includes) {
             const found = this.findInclude(node, obj);
             if (found) {
-                this.addAllFormDefinitions(found, obj);
+                this.addAllFormDefinitions(found, obj, visited);
             }
         }
     }
     findInclude(node, obj) {
-        // assumption: no cyclic includes, includes not found are reported by rule "check_include"
+        // cyclic includes are cut by the callers, includes not found are reported by rule "check_include"
         // todo: how to make sure code is not duplicated here and in rule "check_include" / include graph?
         const expr = node.findFirstExpression(Expressions.IncludeName);
         if (expr === undefined) {
@@ -29315,7 +29322,9 @@ class TypeUtils {
             return false;
         }
         else if (source instanceof basic_1.Integer8Type) {
-            if (target instanceof basic_1.IntegerType || target instanceof basic_1.StringType) {
+            if (target instanceof basic_1.IntegerType
+                || target instanceof basic_1.StringType
+                || target instanceof basic_1.FloatType) {
                 return false;
             }
         }
@@ -31183,7 +31192,11 @@ const _syntax_input_1 = __webpack_require__(/*! ../_syntax_input */ "./node_modu
 // On a character or byte field of fixed length the substring must lie inside the field, else it is
 // a syntax error. Other types, a dynamic offset or length and "*" (undefined) are left alone, as is
 // length 0. The offset is applied first, so the length is compared to what remains after the offset.
+// A substring of a date, time or numeric text field has type n; its bounds are not checked.
 function substringType(context, offset, length, node, input) {
+    if (context instanceof basic_1.DateType || context instanceof basic_1.TimeType || context instanceof basic_1.NumericType) {
+        return numericSubstringType(context, offset, length);
+    }
     if (!(context instanceof basic_1.CharacterType) && !(context instanceof basic_1.HexType)) {
         return context;
     }
@@ -31206,6 +31219,24 @@ function substringType(context, offset, length, node, input) {
     }
     return context;
 }
+function numericSubstringType(context, offset, length) {
+    const hasOffset = offset !== undefined && offset > 0;
+    const hasLength = length !== undefined && length > 0;
+    if (hasOffset === false && hasLength === false) {
+        return context;
+    }
+    let remaining = context instanceof basic_1.DateType ? 8 : context instanceof basic_1.TimeType ? 6 : context.getLength();
+    if (hasOffset) {
+        remaining = remaining - offset;
+    }
+    if (hasLength) {
+        remaining = Math.min(remaining, length);
+    }
+    if (remaining <= 0) {
+        return context;
+    }
+    return new basic_1.NumericType(remaining);
+}
 // A field symbol typed "c" or "x" is generic, but its type is represented with length 1, so the
 // offset or length at "index" is not checked when it directly follows the field symbol
 function isFieldSymbolItself(children, index) {
@@ -41338,6 +41369,16 @@ class Convert {
                 target_1.Target.runSyntax(stampTarget, input);
             }
         }
+        const utclongTarget = node.findExpressionAfterToken("UTCLONG");
+        if ((utclongTarget === null || utclongTarget === void 0 ? void 0 : utclongTarget.get()) instanceof Expressions.Target) {
+            const inline = utclongTarget === null || utclongTarget === void 0 ? void 0 : utclongTarget.findDirectExpression(Expressions.InlineData);
+            if (inline) {
+                inline_data_1.InlineData.runSyntax(inline, input, new basic_1.UTCLongType());
+            }
+            else {
+                target_1.Target.runSyntax(utclongTarget, input);
+            }
+        }
     }
 }
 exports.Convert = Convert;
@@ -50858,6 +50899,8 @@ if (Object.keys(map).length === 0) {
 // -----------------------------------
 class SyntaxLogic {
     constructor(reg, object) {
+        // files the traversal walked into through INCLUDEs, outermost first
+        this.includeStack = [];
         this.reg = reg;
         this.issues = [];
         this.deferred = [];
@@ -50965,11 +51008,20 @@ class SyntaxLogic {
             // walk into INCLUDEs
             if (isStatement && child.get() instanceof Statements.Include) {
                 const file = this.helpers.proc.findInclude(child, this.object);
-                if (file !== undefined && file.getStructure() !== undefined) {
+                // an INCLUDE that leads back to a file already being walked is not followed again
+                const cyclic = file !== undefined && (file.getFilename() === this.currentFile.getFilename()
+                    || this.includeStack.includes(file.getFilename()));
+                if (file !== undefined && file.getStructure() !== undefined && cyclic === false) {
                     const old = this.currentFile;
+                    this.includeStack.push(old.getFilename());
                     this.currentFile = file;
-                    this.traverse(file.getStructure());
-                    this.currentFile = old;
+                    try {
+                        this.traverse(file.getStructure());
+                    }
+                    finally {
+                        this.currentFile = old;
+                        this.includeStack.pop();
+                    }
                 }
             }
             if (isStructure || isStatement) {
@@ -51379,11 +51431,46 @@ exports.ArtifactsABAP = ArtifactsABAP;
 
 "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.CrossIncludeMacros = void 0;
 const _abap_object_1 = __webpack_require__(/*! ../objects/_abap_object */ "./node_modules/@abaplint/core/build/src/objects/_abap_object.js");
 const program_1 = __webpack_require__(/*! ../objects/program */ "./node_modules/@abaplint/core/build/src/objects/program.js");
 const _statement_1 = __webpack_require__(/*! ./2_statements/statements/_statement */ "./node_modules/@abaplint/core/build/src/abap/2_statements/statements/_statement.js");
+const statements_1 = __webpack_require__(/*! ./2_statements/statements */ "./node_modules/@abaplint/core/build/src/abap/2_statements/statements/index.js");
+const Expressions = __importStar(__webpack_require__(/*! ./2_statements/expressions */ "./node_modules/@abaplint/core/build/src/abap/2_statements/expressions/index.js"));
 const expand_macros_1 = __webpack_require__(/*! ./2_statements/expand_macros */ "./node_modules/@abaplint/core/build/src/abap/2_statements/expand_macros.js");
 const include_graph_1 = __webpack_require__(/*! ../utils/include_graph */ "./node_modules/@abaplint/core/build/src/utils/include_graph.js");
 // Macros can be defined in one include and used in a sibling include, connected
@@ -51396,6 +51483,7 @@ class CrossIncludeMacros {
         this.reg = reg;
     }
     run() {
+        this.reparseCyclicPrograms();
         const candidates = this.findCandidates();
         if (candidates.length === 0) {
             return;
@@ -51423,6 +51511,45 @@ class CrossIncludeMacros {
         }
     }
     //////////////////////////////
+    // A program or include in an INCLUDE cycle can be parsed while the include defining its macros is still
+    // being parsed, depending on the order the files were added. Everything is parsed now, so parse it again.
+    reparseCyclicPrograms() {
+        const config = this.reg.getConfig();
+        for (const obj of this.reg.getObjects()) {
+            if (obj instanceof program_1.Program && this.hasUnknown(obj) && this.isInIncludeCycle(obj)) {
+                obj.setDirty();
+                obj.parse(config.getRelease(), config.getSyntaxSetttings().globalMacros, this.reg, config.getLanguageVersion());
+            }
+        }
+    }
+    hasUnknown(prog) {
+        return prog.getABAPFiles().some(f => f.getStatements().some(s => s.get() instanceof _statement_1.Unknown));
+    }
+    // the include graph does not link INCLUDEs of main programs, so follow the INCLUDEs from the program
+    isInIncludeCycle(prog) {
+        var _a;
+        const seen = new Set();
+        const stack = [prog];
+        while (stack.length > 0) {
+            for (const file of stack.pop().getABAPFiles()) {
+                for (const statement of file.getStatements()) {
+                    if (!(statement.get() instanceof statements_1.Include)) {
+                        continue;
+                    }
+                    const name = (_a = statement.findDirectExpression(Expressions.IncludeName)) === null || _a === void 0 ? void 0 : _a.concatTokens();
+                    const included = this.reg.getObject("PROG", name);
+                    if (included === prog) {
+                        return true;
+                    }
+                    else if (included instanceof program_1.Program && seen.has(included) === false) {
+                        seen.add(included);
+                        stack.push(included);
+                    }
+                }
+            }
+        }
+        return false;
+    }
     findCandidates() {
         const ret = [];
         for (const obj of this.reg.getObjects()) {
@@ -63196,6 +63323,7 @@ class ABAPObject extends _abstract_object_1.AbstractObject {
     }
     constructor(name) {
         super(name);
+        this.parsing = false;
         this.parsed = [];
         this.texts = undefined;
         this.rawXMLCache = undefined;
@@ -63204,11 +63332,19 @@ class ABAPObject extends _abstract_object_1.AbstractObject {
         return !!x && x instanceof ABAPObject;
     }
     parse(release, globalMacros, reg, languageVersion = version_1.LanguageVersion.Normal) {
-        if (this.isDirty() === false) {
+        // an INCLUDE that leads back to this object while it is being parsed is not followed again
+        if (this.isDirty() === false || this.parsing === true) {
             return { updated: false, runtime: 0 };
         }
         const abapFiles = this.getFiles().filter(f => f.getFilename().endsWith(".abap"));
-        const result = new abap_parser_1.ABAPParser({ release, globalMacros, reg, languageVersion }).parse(abapFiles);
+        let result;
+        this.parsing = true;
+        try {
+            result = new abap_parser_1.ABAPParser({ release, globalMacros, reg, languageVersion }).parse(abapFiles);
+        }
+        finally {
+            this.parsing = false;
+        }
         this.parsed = result.output;
         this.old = result.issues;
         this.dirty = false;
@@ -72425,7 +72561,7 @@ class Registry {
     }
     static abaplintVersion() {
         // magic, see build script "version.js"
-        return "2.120.77";
+        return "2.120.80";
     }
     getDDICReferences() {
         return this.ddicReferences;
@@ -88174,6 +88310,7 @@ __exportStar(__webpack_require__(/*! ./inline_data_old_versions */ "./node_modul
 __exportStar(__webpack_require__(/*! ./inline_packed_computation */ "./node_modules/@abaplint/core/build/src/rules/inline_packed_computation.js"), exports);
 __exportStar(__webpack_require__(/*! ./intf_referencing_clas */ "./node_modules/@abaplint/core/build/src/rules/intf_referencing_clas.js"), exports);
 __exportStar(__webpack_require__(/*! ./invalid_table_index */ "./node_modules/@abaplint/core/build/src/rules/invalid_table_index.js"), exports);
+__exportStar(__webpack_require__(/*! ./is_assigned_in_loop */ "./node_modules/@abaplint/core/build/src/rules/is_assigned_in_loop.js"), exports);
 __exportStar(__webpack_require__(/*! ./keep_single_parameter_on_one_line */ "./node_modules/@abaplint/core/build/src/rules/keep_single_parameter_on_one_line.js"), exports);
 __exportStar(__webpack_require__(/*! ./keyword_case */ "./node_modules/@abaplint/core/build/src/rules/keyword_case.js"), exports);
 __exportStar(__webpack_require__(/*! ./line_break_multiple_parameters */ "./node_modules/@abaplint/core/build/src/rules/line_break_multiple_parameters.js"), exports);
@@ -88239,6 +88376,7 @@ __exportStar(__webpack_require__(/*! ./prefix_is_current_class */ "./node_module
 __exportStar(__webpack_require__(/*! ./range_row_values */ "./node_modules/@abaplint/core/build/src/rules/range_row_values.js"), exports);
 __exportStar(__webpack_require__(/*! ./reduce_procedural_code */ "./node_modules/@abaplint/core/build/src/rules/reduce_procedural_code.js"), exports);
 __exportStar(__webpack_require__(/*! ./reduce_string_templates */ "./node_modules/@abaplint/core/build/src/rules/reduce_string_templates.js"), exports);
+__exportStar(__webpack_require__(/*! ./redundant_cast */ "./node_modules/@abaplint/core/build/src/rules/redundant_cast.js"), exports);
 __exportStar(__webpack_require__(/*! ./redundant_conversion */ "./node_modules/@abaplint/core/build/src/rules/redundant_conversion.js"), exports);
 __exportStar(__webpack_require__(/*! ./release_idoc */ "./node_modules/@abaplint/core/build/src/rules/release_idoc.js"), exports);
 __exportStar(__webpack_require__(/*! ./remove_descriptions */ "./node_modules/@abaplint/core/build/src/rules/remove_descriptions.js"), exports);
@@ -88923,6 +89061,258 @@ exports.InvalidTableIndex = InvalidTableIndex;
 
 /***/ },
 
+/***/ "./node_modules/@abaplint/core/build/src/rules/is_assigned_in_loop.js"
+/*!****************************************************************************!*\
+  !*** ./node_modules/@abaplint/core/build/src/rules/is_assigned_in_loop.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.IsAssignedInLoop = exports.IsAssignedInLoopConf = void 0;
+const Statements = __importStar(__webpack_require__(/*! ../abap/2_statements/statements */ "./node_modules/@abaplint/core/build/src/abap/2_statements/statements/index.js"));
+const Expressions = __importStar(__webpack_require__(/*! ../abap/2_statements/expressions */ "./node_modules/@abaplint/core/build/src/abap/2_statements/expressions/index.js"));
+const issue_1 = __webpack_require__(/*! ../issue */ "./node_modules/@abaplint/core/build/src/issue.js");
+const _basic_rule_config_1 = __webpack_require__(/*! ./_basic_rule_config */ "./node_modules/@abaplint/core/build/src/rules/_basic_rule_config.js");
+const _abap_rule_1 = __webpack_require__(/*! ./_abap_rule */ "./node_modules/@abaplint/core/build/src/rules/_abap_rule.js");
+const _irule_1 = __webpack_require__(/*! ./_irule */ "./node_modules/@abaplint/core/build/src/rules/_irule.js");
+const edit_helper_1 = __webpack_require__(/*! ../edit_helper */ "./node_modules/@abaplint/core/build/src/edit_helper.js");
+const severity_1 = __webpack_require__(/*! ../severity */ "./node_modules/@abaplint/core/build/src/severity.js");
+class IsAssignedInLoopConf extends _basic_rule_config_1.BasicRuleConfig {
+    constructor() {
+        super(...arguments);
+        this.severity = severity_1.Severity.Warning;
+    }
+}
+exports.IsAssignedInLoopConf = IsAssignedInLoopConf;
+class IsAssignedInLoop extends _abap_rule_1.ABAPRule {
+    constructor() {
+        super(...arguments);
+        this.conf = new IsAssignedInLoopConf();
+    }
+    getMetadata() {
+        return {
+            key: "is_assigned_in_loop",
+            title: "IS ASSIGNED after READ TABLE or ASSIGN in a loop",
+            shortDescription: `Reports IS [NOT] ASSIGNED after a READ TABLE ... ASSIGNING or ASSIGN inside a loop, without UNASSIGN before it`,
+            extendedInformation: `A failed READ TABLE ... ASSIGNING or ASSIGN keeps the previous binding of the field symbol. Inside a
+loop, the field symbol still points to the row of an earlier iteration, so IS ASSIGNED is true
+although the statement failed, and the code goes on with the wrong row.
+
+Add UNASSIGN <fs> before the statement, inside the loop, or check sy-subrc after a READ TABLE.
+The quick fix inserts the UNASSIGN; it is not offered for a field symbol declared inline in
+the statement, or for a chained statement.
+
+Not reported: a check that also reads sy-subrc, an ASSIGN that cannot fail (a static source
+without dereference, table expression or dynamic part), and loops outside of the
+method, form or function module of the statement. See also subrc_after_assign.`,
+            tags: [_irule_1.RuleTag.SingleFile, _irule_1.RuleTag.Quickfix],
+            badExample: `LOOP AT lt_cells INTO ls_cell.
+  READ TABLE lt_styles ASSIGNING <style> WITH KEY id = ls_cell-style.
+  IF <style> IS ASSIGNED.
+    ls_cell-format = <style>-format.
+  ENDIF.
+ENDLOOP.`,
+            goodExample: `LOOP AT lt_cells INTO ls_cell.
+  UNASSIGN <style>.
+  READ TABLE lt_styles ASSIGNING <style> WITH KEY id = ls_cell-style.
+  IF <style> IS ASSIGNED.
+    ls_cell-format = <style>-format.
+  ENDIF.
+ENDLOOP.`,
+        };
+    }
+    getConfig() {
+        return this.conf;
+    }
+    setConfig(conf) {
+        this.conf = conf;
+    }
+    runParsed(file) {
+        const issues = [];
+        const statements = file.getStatements();
+        for (let i = 0; i < statements.length; i++) {
+            const statement = statements[i];
+            const name = this.boundByFailable(statement);
+            if (name === undefined) {
+                continue;
+            }
+            const fs = name.toUpperCase();
+            const loop = this.innermostLoop(i, statements);
+            if (loop === undefined || this.unassignedBefore(loop, i, statements, fs) === true) {
+                continue;
+            }
+            const check = this.nextCheck(i, statements, fs);
+            if (check === undefined) {
+                continue;
+            }
+            const read = statement.get() instanceof Statements.ReadTable;
+            const message = `${read ? "READ TABLE" : "ASSIGN"} in a loop keeps the previous binding of ${name} when it fails, `
+                + `UNASSIGN ${name} before it` + (read ? " or check sy-subrc" : "");
+            const fix = this.buildFix(file, statement, name);
+            issues.push(issue_1.Issue.atStatement(file, check, message, this.getMetadata().key, this.conf.severity, fix));
+        }
+        return issues;
+    }
+    ////////////////
+    /** the field symbol bound by a READ TABLE ... ASSIGNING or an ASSIGN that can fail */
+    boundByFailable(statement) {
+        var _a, _b, _c;
+        const s = statement.get();
+        if (s instanceof Statements.ReadTable) {
+            const target = (_a = statement.findDirectExpression(Expressions.ReadTableTarget)) === null || _a === void 0 ? void 0 : _a.findDirectExpression(Expressions.FSTarget);
+            return (_b = target === null || target === void 0 ? void 0 : target.findFirstExpression(Expressions.FieldSymbol)) === null || _b === void 0 ? void 0 : _b.concatTokens();
+        }
+        else if (s instanceof Statements.Assign) {
+            const source = statement.findDirectExpression(Expressions.AssignSource);
+            if (source === undefined) {
+                return undefined;
+            }
+            const first = source.getFirstToken().getStr().toUpperCase();
+            const failable = first === "COMPONENT"
+                || first === "TABLE"
+                || source.findFirstExpression(Expressions.Dynamic) !== undefined
+                || source.findFirstExpression(Expressions.Dereference) !== undefined
+                || source.findFirstExpression(Expressions.TableExpression) !== undefined
+                || statement.findDirectTokenByText("INCREMENT") !== undefined;
+            if (failable === false) {
+                return undefined;
+            }
+            const target = statement.findDirectExpression(Expressions.FSTarget);
+            return (_c = target === null || target === void 0 ? void 0 : target.findFirstExpression(Expressions.FieldSymbol)) === null || _c === void 0 ? void 0 : _c.concatTokens();
+        }
+        return undefined;
+    }
+    /** index of the innermost LOOP, DO, WHILE or SELECT loop around the statement, in the same block */
+    innermostLoop(index, statements) {
+        let depth = 0;
+        for (let i = index - 1; i >= 0 && this.startsBlock(statements[i]) === false; i--) {
+            if (this.endsLoop(statements[i])) {
+                depth++;
+            }
+            else if (this.startsLoop(statements[i])) {
+                if (depth === 0) {
+                    return i;
+                }
+                depth--;
+            }
+        }
+        return undefined;
+    }
+    unassignedBefore(loop, index, statements, fs) {
+        for (let i = loop + 1; i < index; i++) {
+            if (statements[i].get() instanceof Statements.Unassign && this.mentions(statements[i], fs)) {
+                return true;
+            }
+        }
+        return false;
+    }
+    /** the first IS [NOT] ASSIGNED of the field symbol after the statement, before the end of the loop
+     * or another UNASSIGN or binding of it; a check that also reads sy-subrc is fine */
+    nextCheck(index, statements, fs) {
+        let depth = 0;
+        for (let i = index + 1; i < statements.length && this.startsBlock(statements[i]) === false; i++) {
+            const statement = statements[i];
+            if (this.startsLoop(statement)) {
+                depth++;
+            }
+            else if (this.endsLoop(statement)) {
+                if (depth === 0) {
+                    return undefined;
+                }
+                depth--;
+            }
+            const concat = statement.concatTokens().toUpperCase();
+            if (concat.includes(fs + " IS ASSIGNED") || concat.includes(fs + " IS NOT ASSIGNED")) {
+                return concat.includes("SY-SUBRC") ? undefined : statement;
+            }
+            if ((statement.get() instanceof Statements.Unassign && this.mentions(statement, fs))
+                || this.binds(statement, fs)) {
+                return undefined;
+            }
+        }
+        return undefined;
+    }
+    mentions(statement, fs) {
+        return statement.findAllExpressions(Expressions.FieldSymbol).some(e => e.concatTokens().toUpperCase() === fs);
+    }
+    binds(statement, fs) {
+        return statement.findAllExpressions(Expressions.FSTarget)
+            .some(t => { var _a; return ((_a = t.findFirstExpression(Expressions.FieldSymbol)) === null || _a === void 0 ? void 0 : _a.concatTokens().toUpperCase()) === fs; });
+    }
+    startsLoop(statement) {
+        const s = statement.get();
+        return s instanceof Statements.Loop
+            || s instanceof Statements.Do
+            || s instanceof Statements.While
+            || s instanceof Statements.SelectLoop;
+    }
+    endsLoop(statement) {
+        const s = statement.get();
+        return s instanceof Statements.EndLoop
+            || s instanceof Statements.EndDo
+            || s instanceof Statements.EndWhile
+            || s instanceof Statements.EndSelect;
+    }
+    startsBlock(statement) {
+        const s = statement.get();
+        return s instanceof Statements.MethodImplementation
+            || s instanceof Statements.Form
+            || s instanceof Statements.FunctionModule
+            || s instanceof Statements.Module
+            || s instanceof Statements.EndMethod
+            || s instanceof Statements.EndForm
+            || s instanceof Statements.EndFunction
+            || s instanceof Statements.EndModule;
+    }
+    /** UNASSIGN on a line of its own before the statement, with the same indentation */
+    buildFix(file, statement, name) {
+        if (statement.getColon() !== undefined || statement.findFirstExpression(Expressions.InlineFS) !== undefined) {
+            return undefined;
+        }
+        const start = statement.getStart();
+        return edit_helper_1.EditHelper.insertAt(file, start, `UNASSIGN ${name}.\n` + " ".repeat(start.getCol() - 1));
+    }
+}
+exports.IsAssignedInLoop = IsAssignedInLoop;
+//# sourceMappingURL=is_assigned_in_loop.js.map
+
+/***/ },
+
 /***/ "./node_modules/@abaplint/core/build/src/rules/keep_single_parameter_on_one_line.js"
 /*!******************************************************************************************!*\
   !*** ./node_modules/@abaplint/core/build/src/rules/keep_single_parameter_on_one_line.js ***!
@@ -90622,7 +91012,11 @@ ENDIF.
                 && lastChildren[1].getChildren().length === 1
                 && lastChildren[1].getFirstToken().getStr().startsWith("-") === false
                 && lastChildren[2].getFirstToken().getStr() === ")") {
-                const issue = issue_1.Issue.atToken(file, lastChildren[0].getFirstToken(), "Too many parentheses", this.getMetadata().key, this.conf.severity);
+                const fixText = lastChildren[1].concatTokens();
+                const start = lastChildren[0].getFirstToken().getStart();
+                const end = lastChildren[2].getLastToken().getEnd();
+                const fix = edit_helper_1.EditHelper.replaceRange(file, start, end, fixText);
+                const issue = issue_1.Issue.atToken(file, lastChildren[0].getFirstToken(), "Too many parentheses", this.getMetadata().key, this.conf.severity, fix);
                 issues.push(issue);
             }
         }
@@ -90751,6 +91145,7 @@ const issue_1 = __webpack_require__(/*! ../issue */ "./node_modules/@abaplint/co
 const _abap_rule_1 = __webpack_require__(/*! ./_abap_rule */ "./node_modules/@abaplint/core/build/src/rules/_abap_rule.js");
 const _basic_rule_config_1 = __webpack_require__(/*! ./_basic_rule_config */ "./node_modules/@abaplint/core/build/src/rules/_basic_rule_config.js");
 const _irule_1 = __webpack_require__(/*! ./_irule */ "./node_modules/@abaplint/core/build/src/rules/_irule.js");
+const edit_helper_1 = __webpack_require__(/*! ../edit_helper */ "./node_modules/@abaplint/core/build/src/edit_helper.js");
 class MaxOneMethodParameterPerLineConf extends _basic_rule_config_1.BasicRuleConfig {
 }
 exports.MaxOneMethodParameterPerLineConf = MaxOneMethodParameterPerLineConf;
@@ -90764,7 +91159,7 @@ class MaxOneMethodParameterPerLine extends _abap_rule_1.ABAPRule {
             key: "max_one_method_parameter_per_line",
             title: "Max one method parameter definition per line",
             shortDescription: `Keep max one method parameter description per line`,
-            tags: [_irule_1.RuleTag.SingleFile, _irule_1.RuleTag.Whitespace],
+            tags: [_irule_1.RuleTag.SingleFile, _irule_1.RuleTag.Whitespace, _irule_1.RuleTag.Quickfix],
             badExample: `
 METHODS apps_scope_token
   IMPORTING
@@ -90793,7 +91188,8 @@ METHODS apps_scope_token
                     continue;
                 }
                 if (prev.getFirstToken().getStart().getRow() === p.getFirstToken().getStart().getRow()) {
-                    const issue = issue_1.Issue.atToken(file, prev.getFirstToken(), this.getMetadata().title, this.getMetadata().key, this.conf.severity);
+                    const fix = this.buildFix(file, statement, p);
+                    const issue = issue_1.Issue.atToken(file, prev.getFirstToken(), this.getMetadata().title, this.getMetadata().key, this.conf.severity, fix);
                     issues.push(issue);
                 }
                 prev = p;
@@ -90801,8 +91197,32 @@ METHODS apps_scope_token
         }
         return issues;
     }
+    /** moves the parameter, or its preceding section keyword, to a new line */
+    buildFix(file, statement, param) {
+        const tokens = statement.getTokens();
+        const index = tokens.indexOf(param.getFirstToken());
+        if (index < 1) {
+            return undefined;
+        }
+        let target = tokens[index];
+        let indent = statement.findFirstExpression(Expressions.MethodParam).getFirstToken().getStart().getCol();
+        const before = tokens[index - 1];
+        if (MaxOneMethodParameterPerLine.SECTIONS.includes(before.getStr().toUpperCase())) {
+            if (index < 2) {
+                return undefined;
+            }
+            target = before;
+            indent = tokens.find(t => MaxOneMethodParameterPerLine.SECTIONS.includes(t.getStr().toUpperCase())).getStart().getCol();
+        }
+        const previous = tokens[tokens.indexOf(target) - 1];
+        if (previous.getEnd().getRow() !== target.getStart().getRow()) {
+            return undefined;
+        }
+        return edit_helper_1.EditHelper.replaceRange(file, previous.getEnd(), target.getStart(), "\n" + " ".repeat(indent - 1));
+    }
 }
 exports.MaxOneMethodParameterPerLine = MaxOneMethodParameterPerLine;
+MaxOneMethodParameterPerLine.SECTIONS = ["IMPORTING", "EXPORTING", "CHANGING", "RETURNING"];
 //# sourceMappingURL=max_one_method_parameter_per_line.js.map
 
 /***/ },
@@ -92214,6 +92634,8 @@ const _abap_rule_1 = __webpack_require__(/*! ./_abap_rule */ "./node_modules/@ab
 const sequential_blank_1 = __webpack_require__(/*! ./sequential_blank */ "./node_modules/@abaplint/core/build/src/rules/sequential_blank.js");
 const Statements = __importStar(__webpack_require__(/*! ../abap/2_statements/statements */ "./node_modules/@abaplint/core/build/src/abap/2_statements/statements/index.js"));
 const _irule_1 = __webpack_require__(/*! ./_irule */ "./node_modules/@abaplint/core/build/src/rules/_irule.js");
+const edit_helper_1 = __webpack_require__(/*! ../edit_helper */ "./node_modules/@abaplint/core/build/src/edit_helper.js");
+const position_1 = __webpack_require__(/*! ../position */ "./node_modules/@abaplint/core/build/src/position.js");
 var NewlineLogic;
 (function (NewlineLogic) {
     NewlineLogic["Exact"] = "exact";
@@ -92243,7 +92665,7 @@ class NewlineBetweenMethods extends _abap_rule_1.ABAPRule {
             key: "newline_between_methods",
             title: "New line between methods",
             shortDescription: `Checks for newlines between method implementations.`,
-            tags: [_irule_1.RuleTag.Whitespace, _irule_1.RuleTag.SingleFile],
+            tags: [_irule_1.RuleTag.Whitespace, _irule_1.RuleTag.Quickfix, _irule_1.RuleTag.SingleFile],
         };
     }
     getMessage() {
@@ -92272,18 +92694,35 @@ class NewlineBetweenMethods extends _abap_rule_1.ABAPRule {
                 continue;
             }
             let counter = 0;
-            while (sequential_blank_1.SequentialBlank.isBlankOrWhitespace(rows[nextRow]) && (counter <= this.conf.count + 1)) {
+            while (sequential_blank_1.SequentialBlank.isBlankOrWhitespace(rows[nextRow])) {
                 counter++;
                 nextRow++;
             }
             if ((counter !== this.conf.count && this.conf.logic === NewlineLogic.Exact)
                 || (counter >= this.conf.count && this.conf.logic === NewlineLogic.Less)
                 || counter === 0) {
-                issues.push(issue_1.Issue.atStatement(file, statement, this.getMessage(), this.getMetadata().key, this.conf.severity));
+                const fix = this.buildFix(file, rows, statement.getStart().getRow(), counter);
+                issues.push(issue_1.Issue.atStatement(file, statement, this.getMessage(), this.getMetadata().key, this.conf.severity, fix));
             }
         }
         return issues;
     }
+    buildFix(file, rows, row, blanks) {
+        let target = this.conf.count;
+        if (this.conf.logic === NewlineLogic.Less) {
+            target = blanks === 0 ? 1 : this.conf.count - 1;
+        }
+        if (blanks < target) {
+            const end = new position_1.Position(row, rows[row - 1].length + 1);
+            return edit_helper_1.EditHelper.insertAt(file, end, "\n".repeat(target - blanks));
+        }
+        else if (blanks > target) {
+            const start = new position_1.Position(row + target, rows[row + target - 1].length + 1);
+            const end = new position_1.Position(row + blanks, rows[row + blanks - 1].length + 1);
+            return edit_helper_1.EditHelper.deleteRange(file, start, end);
+        }
+        return undefined;
+    }
     isConfigurationValid() {
         if (this.conf.count < 1 || (this.conf.count === 1 && this.conf.logic === NewlineLogic.Less)) {
             return false;
@@ -97208,7 +97647,7 @@ class PreferPragmas extends _abap_rule_1.ABAPRule {
         return {
             key: "prefer_pragmas",
             title: "prefer pragmas over pseudo comments ",
-            shortDescription: `prefer pragmas over pseudo comments `,
+            shortDescription: `prefer pragmas over pseudo comments`,
             extendedInformation: `https://github.com/SAP/styleguides/blob/main/clean-abap/CleanABAP.md#prefer-pragmas-to-pseudo-comments`,
             tags: [_irule_1.RuleTag.SingleFile, _irule_1.RuleTag.Styleguide],
             badExample: `DATA foo1 TYPE i. "#EC NEEDED`,
@@ -98725,6 +99164,173 @@ exports.ReduceStringTemplates = ReduceStringTemplates;
 
 /***/ },
 
+/***/ "./node_modules/@abaplint/core/build/src/rules/redundant_cast.js"
+/*!***********************************************************************!*\
+  !*** ./node_modules/@abaplint/core/build/src/rules/redundant_cast.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.RedundantCast = exports.RedundantCastConf = void 0;
+const Expressions = __importStar(__webpack_require__(/*! ../abap/2_statements/expressions */ "./node_modules/@abaplint/core/build/src/abap/2_statements/expressions/index.js"));
+const _reference_1 = __webpack_require__(/*! ../abap/5_syntax/_reference */ "./node_modules/@abaplint/core/build/src/abap/5_syntax/_reference.js");
+const syntax_1 = __webpack_require__(/*! ../abap/5_syntax/syntax */ "./node_modules/@abaplint/core/build/src/abap/5_syntax/syntax.js");
+const nodes_1 = __webpack_require__(/*! ../abap/nodes */ "./node_modules/@abaplint/core/build/src/abap/nodes/index.js");
+const _typed_identifier_1 = __webpack_require__(/*! ../abap/types/_typed_identifier */ "./node_modules/@abaplint/core/build/src/abap/types/_typed_identifier.js");
+const basic_1 = __webpack_require__(/*! ../abap/types/basic */ "./node_modules/@abaplint/core/build/src/abap/types/basic/index.js");
+const issue_1 = __webpack_require__(/*! ../issue */ "./node_modules/@abaplint/core/build/src/issue.js");
+const _abap_object_1 = __webpack_require__(/*! ../objects/_abap_object */ "./node_modules/@abaplint/core/build/src/objects/_abap_object.js");
+const _basic_rule_config_1 = __webpack_require__(/*! ./_basic_rule_config */ "./node_modules/@abaplint/core/build/src/rules/_basic_rule_config.js");
+const _irule_1 = __webpack_require__(/*! ./_irule */ "./node_modules/@abaplint/core/build/src/rules/_irule.js");
+const edit_helper_1 = __webpack_require__(/*! ../edit_helper */ "./node_modules/@abaplint/core/build/src/edit_helper.js");
+const version_1 = __webpack_require__(/*! ../version */ "./node_modules/@abaplint/core/build/src/version.js");
+class RedundantCastConf extends _basic_rule_config_1.BasicRuleConfig {
+}
+exports.RedundantCastConf = RedundantCastConf;
+class RedundantCast {
+    constructor() {
+        this.conf = new RedundantCastConf();
+    }
+    getMetadata() {
+        return {
+            key: "redundant_cast",
+            title: "Redundant Cast",
+            shortDescription: `Find redundant CAST expressions`,
+            extendedInformation: `Reports CAST expressions whose operand is already a reference to the cast's target class or interface.`,
+            tags: [_irule_1.RuleTag.Quickfix],
+            badExample: `DATA ref TYPE REF TO lcl.
+DATA(copy) = CAST lcl( ref ).`,
+            goodExample: `DATA ref TYPE REF TO lcl.
+DATA(copy) = ref.`,
+        };
+    }
+    getConfig() {
+        return this.conf;
+    }
+    setConfig(conf) {
+        this.conf = conf;
+    }
+    initialize(reg) {
+        this.reg = reg;
+        return this;
+    }
+    run(obj) {
+        var _a;
+        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 cast of structure.findAllExpressionsRecursive(Expressions.Cast)) {
+                const typeExpression = cast.findDirectExpression(Expressions.TypeNameOrInfer);
+                const source = cast.findDirectExpression(Expressions.Source);
+                if (typeExpression === undefined
+                    || source === undefined
+                    || cast.findDirectExpression(Expressions.Let) !== undefined) {
+                    continue;
+                }
+                const scope = syntax.spaghetti.lookupPosition(cast.getFirstToken().getStart(), file.getFilename());
+                if (scope === undefined) {
+                    continue;
+                }
+                const target = (_a = scope.getData().references.find(reference => reference.referenceType === _reference_1.ReferenceType.InferredType
+                    && reference.position.getStart().equals(typeExpression.getFirstToken().getStart()))) === null || _a === void 0 ? void 0 : _a.resolved;
+                if (!(target instanceof _typed_identifier_1.TypedIdentifier)) {
+                    continue;
+                }
+                const operandType = this.sourceType(source, scope);
+                if (operandType === undefined || this.sameType(target.getType(), operandType) === false) {
+                    continue;
+                }
+                const closingParen = cast.getDirectTokens().find(token => token.getStr() === ")" && token.getStart().isAfter(source.getLastToken().getStart()));
+                if (closingParen === undefined) {
+                    continue;
+                }
+                const fix = edit_helper_1.EditHelper.replaceRange(file, cast.getFirstToken().getStart(), closingParen.getEnd(), source.concatTokens());
+                issues.push(issue_1.Issue.atRange(file, cast.getFirstToken().getStart(), closingParen.getEnd(), "Redundant CAST expression", this.getMetadata().key, this.conf.severity, fix));
+            }
+        }
+        return issues;
+    }
+    /** The type of a plain variable or attribute, undefined for anything more complex */
+    sourceType(source, scope) {
+        var _a;
+        const children = source.getChildren();
+        if (children.length !== 1 || !(children[0].get() instanceof Expressions.FieldChain)) {
+            return undefined;
+        }
+        const fieldChain = children[0];
+        const last = fieldChain.getLastChild();
+        if (!(last instanceof nodes_1.ExpressionNode)
+            || !(last.get() instanceof Expressions.SourceField || last.get() instanceof Expressions.AttributeName)) {
+            return undefined;
+        }
+        const operand = (_a = scope.getData().references.find(reference => reference.referenceType === _reference_1.ReferenceType.DataReadReference
+            && reference.position.getStart().equals(last.getFirstToken().getStart()))) === null || _a === void 0 ? void 0 : _a.resolved;
+        if (!(operand instanceof _typed_identifier_1.TypedIdentifier)) {
+            return undefined;
+        }
+        return operand.getType();
+    }
+    sameType(first, second) {
+        if (!(first instanceof basic_1.ObjectReferenceType) || !(second instanceof basic_1.ObjectReferenceType)) {
+            return false;
+        }
+        return first.getIdentifierName().toUpperCase() === second.getIdentifierName().toUpperCase();
+    }
+}
+exports.RedundantCast = RedundantCast;
+//# sourceMappingURL=redundant_cast.js.map
+
+/***/ },
+
 /***/ "./node_modules/@abaplint/core/build/src/rules/redundant_conversion.js"
 /*!*****************************************************************************!*\
   !*** ./node_modules/@abaplint/core/build/src/rules/redundant_conversion.js ***!
@@ -98930,9 +99536,9 @@ text = text.`,
                 context = this.infer(context, this.sourceType(child, scope));
             }
             else if (child.get() instanceof Expressions.ArithOperator) {
-                if (child.concatTokens() === "**") {
-                    context = new basic_1.FloatType();
-                }
+                // the CONV sets the calculation type of arithmetic expressions, only redundant
+                // if the assignment target has the same type, see arithmeticUsesSameTargetType
+                return undefined;
             }
             else {
                 return undefined;
@@ -107611,7 +108217,6 @@ const position_1 = __webpack_require__(/*! ../position */ "./node_modules/@abapl
 const issue_1 = __webpack_require__(/*! ../issue */ "./node_modules/@abaplint/core/build/src/issue.js");
 const _abap_object_1 = __webpack_require__(/*! ../objects/_abap_object */ "./node_modules/@abaplint/core/build/src/objects/_abap_object.js");
 const severity_1 = __webpack_require__(/*! ../severity */ "./node_modules/@abaplint/core/build/src/severity.js");
-// todo, check for cycles/circular dependencies, method findTop
 // todo, add configurable error for multiple use includes
 const FMXXINCLUDE = /^(\/\w+\/)?L.+XX$/;
 function getABAPObjects(reg) {
@@ -107647,11 +108252,33 @@ class Graph {
         }
         this.edges[from.filename].push(toFilename);
     }
-    findTop(filename) {
+    // true if "to" is reachable from "from" via the edges, "from" itself counts
+    isReachable(from, to) {
+        const seen = new Set();
+        const stack = [from];
+        while (stack.length > 0) {
+            const current = stack.pop();
+            if (current === to) {
+                return true;
+            }
+            if (seen.has(current) === false) {
+                seen.add(current);
+                stack.push(...this.edges[current] || []);
+            }
+        }
+        return false;
+    }
+    findTop(filename, path = new Set()) {
         const ret = [];
+        path.add(filename);
         for (const to of this.edges[filename] || []) {
-            ret.push(...this.findTop(to));
+            // an INCLUDE cycle, the file is already on the current path
+            if (path.has(to)) {
+                continue;
+            }
+            ret.push(...this.findTop(to, path));
         }
+        path.delete(filename);
         if (ret.length === 0) {
             const found = this.findVertexByFilename(filename);
             if (found !== undefined) {
@@ -107706,6 +108333,7 @@ class IncludeGraph {
     ///////////////////////////////
     build() {
         this.addVertices();
+        const included = [];
         for (const o of getABAPObjects(this.reg)) {
             for (const f of o.getABAPFiles()) {
                 if (f.getFilename().includes(".prog.screen_") || f.getFilename().includes(".fugr.screen_")) {
@@ -107736,11 +108364,19 @@ class IncludeGraph {
                         }
                         else {
                             this.graph.addEdge(found, f.getFilename());
+                            included.push({ file: f, statement: s, vertex: found, name });
                         }
                     }
                 }
             }
         }
+        // cycles are not followed, but ABAP rejects them, so report them
+        for (const i of included) {
+            if (this.graph.isReachable(i.file.getFilename(), i.vertex.filename)) {
+                const issue = issue_1.Issue.atStatement(i.file, i.statement, "Cyclic INCLUDE, " + i.name, new check_include_1.CheckInclude().getMetadata().key, this.severity);
+                this.issues.push(issue);
+            }
+        }
     }
     addVertices() {
         for (const o of getABAPObjects(this.reg)) {
@@ -112454,7 +113090,7 @@ const chunk_1 = __webpack_require__(/*! ../chunk */ "./node_modules/@abaplint/tr
 class ParameterNameTranspiler {
     transpile(node, traversal) {
         const nameToken = node.getFirstToken();
-        const name = traversal_1.Traversal.escapeNamespace(nameToken.getStr().toLowerCase());
+        const name = traversal_1.Traversal.escapeNamespace(nameToken.getStr().toLowerCase().replace(/^!/, ""));
         return new chunk_1.Chunk().append(name, nameToken, traversal);
     }
 }
@@ -112478,7 +113114,7 @@ const chunk_1 = __webpack_require__(/*! ../chunk */ "./node_modules/@abaplint/tr
 class ParameterSTranspiler {
     transpile(node, traversal) {
         const nameToken = node.findDirectExpression(core_1.Expressions.ParameterName)?.getFirstToken();
-        const name = nameToken?.getStr().toLowerCase();
+        const name = nameToken?.getStr().toLowerCase().replace(/^!/, "");
         const source = traversal.traverse(node.findDirectExpression(core_1.Expressions.Source));
         return new chunk_1.Chunk().append(name + ": ", nameToken || node, traversal).appendChunk(source);
     }
@@ -112503,7 +113139,7 @@ const chunk_1 = __webpack_require__(/*! ../chunk */ "./node_modules/@abaplint/tr
 class ParameterTTranspiler {
     transpile(node, traversal) {
         const nameToken = node.findDirectExpression(core_1.Expressions.ParameterName)?.getFirstToken();
-        const name = nameToken?.getStr();
+        const name = nameToken?.getStr().replace(/^!/, "");
         const source = traversal.traverse(node.findDirectExpression(core_1.Expressions.Target));
         return new chunk_1.Chunk().append(name + ": ", nameToken || node, traversal).appendChunk(source);
     }
@@ -119607,46 +120243,34 @@ exports.ConvertTranspiler = void 0;
 const abaplint = __importStar(__webpack_require__(/*! @abaplint/core */ "./node_modules/@abaplint/core/build/src/index.js"));
 const expressions_1 = __webpack_require__(/*! ../expressions */ "./node_modules/@abaplint/transpiler/build/src/expressions/index.js");
 const chunk_1 = __webpack_require__(/*! ../chunk */ "./node_modules/@abaplint/transpiler/build/src/chunk.js");
+const OPERANDS = ["date", "time", "stamp", "utclong", "zone", "daylight"];
 class ConvertTranspiler {
     transpile(node, traversal) {
-        const source = [];
-        const target = [];
-        const date = node.findExpressionAfterToken("DATE");
-        if (date?.get() instanceof abaplint.Expressions.Source) {
-            source.push("date: " + new expressions_1.SourceTranspiler().transpile(date, traversal).getCode());
-        }
-        else if (date?.get() instanceof abaplint.Expressions.Target) {
-            target.push("date: " + new expressions_1.TargetTranspiler().transpile(date, traversal).getCode());
-        }
-        const time = node.findExpressionAfterToken("TIME");
-        if (time?.get() instanceof abaplint.Expressions.Source) {
-            source.push("time: " + new expressions_1.SourceTranspiler().transpile(time, traversal).getCode());
-        }
-        else if (time?.get() instanceof abaplint.Expressions.Target) {
-            target.push("time: " + new expressions_1.TargetTranspiler().transpile(time, traversal).getCode());
-        }
-        const stamp = node.findExpressionAfterToken("STAMP");
-        if (stamp?.get() instanceof abaplint.Expressions.Source) {
-            source.push("stamp: " + new expressions_1.SourceTranspiler().transpile(stamp, traversal).getCode());
-        }
-        else if (stamp?.get() instanceof abaplint.Expressions.Target) {
-            target.push("stamp: " + new expressions_1.TargetTranspiler().transpile(stamp, traversal).getCode());
-        }
-        const utclong = node.findExpressionAfterToken("UTCLONG");
-        if (utclong?.get() instanceof abaplint.Expressions.Source) {
-            source.push("utclong: " + new expressions_1.SourceTranspiler().transpile(utclong, traversal).getCode());
-        }
-        else if (utclong?.get() instanceof abaplint.Expressions.Target) {
-            target.push("utclong: " + new expressions_1.TargetTranspiler().transpile(utclong, traversal).getCode());
-        }
-        const zone = node.findExpressionAfterToken("ZONE");
-        if (zone?.get() instanceof abaplint.Expressions.Source) {
-            source.push("zone: " + new expressions_1.SourceTranspiler().transpile(zone, traversal).getCode());
-        }
-        else if (zone?.get() instanceof abaplint.Expressions.Target) {
-            target.push("zone: " + new expressions_1.TargetTranspiler().transpile(zone, traversal).getCode());
+        const source = {};
+        const target = {};
+        // each operand is named by the keyword in front of it; "TIME" after "SAVING" is the
+        // daylight saving flag, which may come before or after the TIME operand
+        const children = node.getChildren();
+        for (let i = 1; i < children.length; i++) {
+            const child = children[i];
+            if (!(child instanceof abaplint.Nodes.ExpressionNode)) {
+                continue;
+            }
+            const keyword = children[i - 1].concatTokens().toUpperCase();
+            const before = i >= 2 ? children[i - 2].concatTokens().toUpperCase() : "";
+            const name = keyword === "TIME" && before === "SAVING" ? "daylight" : keyword.toLowerCase();
+            if (OPERANDS.includes(name) === false || source[name] !== undefined || target[name] !== undefined) {
+                continue;
+            }
+            if (child.get() instanceof abaplint.Expressions.Source) {
+                source[name] = new expressions_1.SourceTranspiler().transpile(child, traversal).getCode();
+            }
+            else if (child.get() instanceof abaplint.Expressions.Target) {
+                target[name] = new expressions_1.TargetTranspiler().transpile(child, traversal).getCode();
+            }
         }
-        return new chunk_1.Chunk(`abap.statements.convert({${source.join(",")}}, {${target.join(",")}});`);
+        const list = (operands) => OPERANDS.filter(n => operands[n] !== undefined).map(n => n + ": " + operands[n]).join(",");
+        return new chunk_1.Chunk(`abap.statements.convert({${list(source)}}, {${list(target)}});`);
     }
 }
 exports.ConvertTranspiler = ConvertTranspiler;
@@ -130453,7 +131077,7 @@ class Traversal {
         return false;
     }
     prefixAndName(t, filename) {
-        let name = t.getStr().toLowerCase();
+        let name = t.getStr().toLowerCase().replace(/^!/, "");
         if (filename && this.getCurrentObject().getABAPFileByName(filename) === undefined) {
             // the prefix is from a different object
             const file = this.reg.getFileByName(filename);

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@abaplint/transpiler-cli",
-  "version": "2.14.5",
+  "version": "2.14.7",
   "description": "Transpiler - Command Line Interface",
   "funding": "https://github.com/sponsors/larshp",
   "bin": {
@@ -27,8 +27,8 @@
   "author": "abaplint",
   "license": "MIT",
   "devDependencies": {
-    "@abaplint/core": "^2.120.77",
-    "@abaplint/transpiler": "^2.14.5",
+    "@abaplint/core": "^2.120.80",
+    "@abaplint/transpiler": "^2.14.7",
     "@types/glob": "^9.0.0",
     "@types/node": "^26.6.2",
     "@types/progress": "^2.0.7",