@abaplint/core 2.109.2 → 2.109.3

package/build/abaplint.d.ts

@@ -3221,10 +3221,14 @@ declare interface IMacroReferences {
     addReference(ref: IFilenameAndToken): void;
     listDefinitionsByFile(filename: string): Token[];
     listUsagesbyMacro(filename: string, token: Token): IFilenameAndToken[];
-    getDefinitionPosition(filename: string, token: Token): {
+    getDefinitionRange(filename: string, token: Token): {
         start: Position;
         end: Position;
     } | undefined;
+    findDefinitionByUsage(filename: string, token: Token): {
+        filename: string;
+        token: Token;
+    } | undefined;
     clear(filename: string): void;
 }
 

package/build/src/lsp/_lookup.js

@@ -12,6 +12,7 @@ const _reference_1 = require("../abap/5_syntax/_reference");
 const _builtin_1 = require("../abap/5_syntax/_builtin");
 const _scope_type_1 = require("../abap/5_syntax/_scope_type");
 const types_1 = require("../abap/types");
+const _statement_1 = require("../abap/2_statements/statements/_statement");
 class LSPLookup {
     static lookup(cursor, reg, obj) {
         var _a, _b;
@@ -128,6 +129,18 @@ class LSPLookup {
                 scope: bottomScope,
             };
         }
+        if (cursor.snode.get() instanceof _statement_1.MacroCall) {
+            const macroDefinition = reg.getMacroReferences().findDefinitionByUsage(cursor.identifier.getFilename(), cursor.snode.getFirstToken());
+            if (macroDefinition) {
+                return {
+                    hover: "Macro Call",
+                    definition: {
+                        uri: macroDefinition === null || macroDefinition === void 0 ? void 0 : macroDefinition.filename,
+                        range: _lsp_utils_1.LSPUtils.tokenToRange(macroDefinition.token),
+                    },
+                };
+            }
+        }
         if (hoverValue !== "") {
             return { hover: hoverValue, scope: bottomScope };
         }

package/build/src/lsp/references.js

@@ -30,6 +30,7 @@ class References {
         const locs = this.search(lookup.definitionId, lookup.scope);
         return locs.map(_lsp_utils_1.LSPUtils.identiferToLocation);
     }
+    ////////////////////////////////////////////
     // todo, cleanup this mehtod, some of the method parameters are not used anymore?
     search(identifier, node, exitAfterFound = false, removeDuplicates = true) {
         let ret = [];
@@ -57,7 +58,6 @@ class References {
             return ret;
         }
     }
-    ////////////////////////////////////////////
     removeDuplicates(arr) {
         const values = {};
         return arr.filter(item => {

package/build/src/macro_references.js

@@ -15,7 +15,7 @@ class MacroReferences {
         }
         this.definitions[ref.filename].push({ token: ref.token, start, end });
     }
-    getDefinitionPosition(filename, token) {
+    getDefinitionRange(filename, token) {
         for (const d of this.definitions[filename] || []) {
             if (d.token.getStart().equals(token.getStart())) {
                 return { start: d.start, end: d.end };
@@ -50,6 +50,22 @@ class MacroReferences {
         delete this.definitions[filename];
         delete this.references[filename];
     }
+    findDefinitionByUsage(filename, token) {
+        const tokenStr = token.getStr().toUpperCase();
+        for (const ref of this.references[filename] || []) {
+            if (ref.token.getStart().equals(token.getStart())) {
+                for (const d of this.definitions[ref.filename] || []) {
+                    if (d.token.getStr().toUpperCase() === tokenStr) {
+                        return {
+                            filename: ref.filename,
+                            token: d.token,
+                        };
+                    }
+                }
+            }
+        }
+        return undefined;
+    }
 }
 exports.MacroReferences = MacroReferences;
 //# sourceMappingURL=macro_references.js.map

package/build/src/registry.js

@@ -67,7 +67,7 @@ class Registry {
     }
     static abaplintVersion() {
         // magic, see build script "version.sh"
-        return "2.109.2";
+        return "2.109.3";
     }
     getDDICReferences() {
         return this.ddicReferences;

package/build/src/rules/unused_macros.js

@@ -61,7 +61,7 @@ foobar2.`,
                 const usages = references.listUsagesbyMacro(file.getFilename(), macroToken);
                 if (usages.length === 0 && ((_a = this.conf.skipNames) === null || _a === void 0 ? void 0 : _a.includes(macroToken.getStr().toUpperCase())) === false) {
                     const message = "Unused macro definition: " + macroToken.getStr();
-                    const pos = references.getDefinitionPosition(file.getFilename(), macroToken);
+                    const pos = references.getDefinitionRange(file.getFilename(), macroToken);
                     const fix = edit_helper_1.EditHelper.deleteRange(file, pos.start, pos.end);
                     result.push(issue_1.Issue.atToken(file, macroToken, message, this.getMetadata().key, this.conf.severity, fix));
                 }

package/package.json

@@ -1,6 +1,6 @@
 {
     "name": "@abaplint/core",
-    "version": "2.109.2",
+    "version": "2.109.3",
     "description": "abaplint - Core API",
     "main": "build/src/index.js",
     "typings": "build/abaplint.d.ts",