fix: if multiple nodes don't have a name (should not happen, just a s…

…afeguard), don't report errors
Safe-DS · Sep 21, 2023 · ce59c33 · ce59c33
1 parent 33ed9b1
commit ce59c33
Show file tree

Hide file tree

Showing 2 changed files with 7 additions and 1 deletion.
diff --git a/src/language/ast/shortcuts.ts b/src/language/ast/shortcuts.ts
@@ -31,12 +31,14 @@ import { stream } from 'langium';
 
 export const annotationCallsOrEmpty = function (node: SdsAnnotatedObject | undefined): SdsAnnotationCall[] {
     if (!node) {
+        /* c8 ignore next 2 */
         return [];
     }
 
     if (isSdsDeclaration(node)) {
         return node?.annotationCallList?.annotationCalls ?? node?.annotationCalls ?? [];
     } else {
+        /* c8 ignore next 2 */
         return node?.annotationCalls ?? [];
     }
 };

diff --git a/src/language/validation/names.ts b/src/language/validation/names.ts
@@ -256,7 +256,11 @@ const namesMustBeUnique = (
     const knownNames = new Set<string>();
 
     for (const node of nodes) {
-        const name = node.name ?? '';
+        const name = node.name;
+        if (!name) {
+            continue;
+        }
+
         if (knownNames.has(name)) {
             accept('error', createMessage(name), {
                 node,