@abaplint/core 2.85.42 → 2.85.43

package/build/abaplint.d.ts

@@ -805,6 +805,7 @@ declare class ClassDefinition_3 extends Identifier implements IClassDefinition {
     private readonly implementing;
     private readonly testing;
     private readonly abstract;
+    private readonly sharedMemory;
     private aliases;
     constructor(node: StructureNode, filename: string, scope: CurrentScope);
     getFriends(): string[];
@@ -819,6 +820,7 @@ declare class ClassDefinition_3 extends Identifier implements IClassDefinition {
     getAliases(): IAliases;
     isForTesting(): boolean;
     isAbstract(): boolean;
+    isSharedMemory(): boolean;
     private findSuper;
     private findFriends;
     private addReference;
@@ -2553,6 +2555,7 @@ export declare interface IClassDefinition extends IInterfaceDefinition {
     isFinal(): boolean;
     isForTesting(): boolean;
     isAbstract(): boolean;
+    isSharedMemory(): boolean;
     getFriends(): string[];
 }
 
@@ -2882,6 +2885,7 @@ declare interface InfoClassDefinition extends InfoInterfaceDefinition {
     isFinal: boolean;
     interfaces: readonly InfoImplementing[];
     isForTesting: boolean;
+    isSharedMemory: boolean;
 }
 
 declare interface InfoClassImplementation {

package/build/src/abap/4_file_information/abap_file_information.js

@@ -144,6 +144,7 @@ class ABAPFileInformation {
                 interfaces: this.getImplementing(found),
                 isForTesting: concat.includes(" FOR TESTING"),
                 isAbstract: concat.includes(" ABSTRACT"),
+                isSharedMemory: concat.includes(" SHARED MEMORY ENABLED"),
                 isFinal: found.findFirstExpression(Expressions.ClassFinal) !== undefined,
                 aliases,
                 attributes,

package/build/src/abap/5_syntax/statements/catch.js

@@ -10,9 +10,10 @@ const _reference_1 = require("../_reference");
 class Catch {
     runSyntax(node, scope, filename) {
         var _a, _b;
+        const names = new Set();
         for (const c of node.findDirectExpressions(Expressions.ClassName)) {
             const token = c.getFirstToken();
-            const className = token.getStr();
+            const className = token.getStr().toUpperCase();
             const found = scope.existsObject(className);
             if (found.found === true && found.id) {
                 scope.addReference(token, found.id, found.type, filename);
@@ -24,6 +25,10 @@ class Catch {
             else {
                 throw new Error("CATCH, unknown class " + className);
             }
+            if (names.has(className)) {
+                throw new Error("Duplicate class name in CATCH: " + className);
+            }
+            names.add(className);
         }
         const target = node.findDirectExpression(Expressions.Target);
         const firstClassName = (_a = node.findDirectExpression(Expressions.ClassName)) === null || _a === void 0 ? void 0 : _a.getFirstToken().getStr();

package/build/src/abap/types/class_definition.js

@@ -43,6 +43,7 @@ class ClassDefinition extends _identifier_1.Identifier {
         scope.pop(node.getLastToken().getEnd());
         const concat = this.node.findFirstStatement(Statements.ClassDefinition).concatTokens().toUpperCase();
         this.testing = concat.includes(" FOR TESTING");
+        this.sharedMemory = concat.includes(" SHARED MEMORY ENABLED");
         this.abstract = concat.includes(" ABSTRACT");
     }
     getFriends() {
@@ -81,6 +82,9 @@ class ClassDefinition extends _identifier_1.Identifier {
     isAbstract() {
         return this.abstract;
     }
+    isSharedMemory() {
+        return this.sharedMemory;
+    }
     /*
       public getEvents() {
       }

package/build/src/registry.js

@@ -68,7 +68,7 @@ class Registry {
     }
     static abaplintVersion() {
         // magic, see build script "version.sh"
-        return "2.85.42";
+        return "2.85.43";
     }
     getDDICReferences() {
         return this.references;

package/build/src/rules/cyclic_oo.js

@@ -15,6 +15,8 @@ class CyclicOOConf extends _basic_rule_config_1.BasicRuleConfig {
          * @uniqueItems true
         */
         this.skip = [];
+        /** Skips shared memory enabled classes*/
+        this.skipSharedMemory = true;
     }
 }
 exports.CyclicOOConf = CyclicOOConf;
@@ -41,16 +43,20 @@ class CyclicOO {
         }
     }
     initialize(reg) {
+        var _a;
         this.reg = reg;
         this.edges = {};
         for (const obj of this.reg.getObjectsByType("CLAS")) {
             const name = obj.getName().toUpperCase();
-            if (!(obj instanceof _abap_object_1.ABAPObject)) {
+            if (!(obj instanceof objects_1.Class)) {
                 continue;
             }
             else if (this.conf.skip.indexOf(name) >= 0) {
                 continue;
             }
+            else if (this.conf.skipSharedMemory === true && ((_a = obj.getClassDefinition()) === null || _a === void 0 ? void 0 : _a.isSharedMemory) === true) {
+                continue;
+            }
             this.buildEdges(name, new syntax_1.SyntaxLogic(this.reg, obj).run().spaghetti.getTop());
         }
         for (const obj of this.reg.getObjectsByType("INTF")) {

package/build/src/rules/use_new.js

@@ -59,6 +59,9 @@ Applicable from v740sp02 and up`,
                 else if (statement.findDirectExpression(expressions_1.ParameterListExceptions)) {
                     continue;
                 }
+                else if (statement.findDirectTokenByText("AREA")) {
+                    continue;
+                }
                 const target = ((_a = statement.findDirectExpression(expressions_1.Target)) === null || _a === void 0 ? void 0 : _a.concatTokens()) + "->";
                 if (statement.concatTokens().includes(target)) {
                     continue;

package/package.json

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