Skip to content

Commit 8fe025e

Browse files
authored
Merge pull request #13384 from tamasvajk/fix/standalone-explicit-interface-error
C#: Improve error message for missing explicit interface implementation
2 parents 357542a + 0f75449 commit 8fe025e

File tree

1 file changed

+6
-1
lines changed
  • csharp/extractor/Semmle.Extraction.CSharp/Entities

1 file changed

+6
-1
lines changed

csharp/extractor/Semmle.Extraction.CSharp/Entities/Method.cs

Lines changed: 6 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -243,7 +243,12 @@ public static void AddExplicitInterfaceQualifierToId(Context cx, EscapingTextWri
243243
if (methodKind == MethodKind.ExplicitInterfaceImplementation)
244244
{
245245
// Retrieve the original method kind
246-
methodKind = methodDecl.ExplicitInterfaceImplementations.Select(m => m.MethodKind).FirstOrDefault();
246+
if (methodDecl.ExplicitInterfaceImplementations.IsEmpty)
247+
{
248+
throw new InternalError(methodDecl, "Couldn't get the original method kind for an explicit interface implementation");
249+
}
250+
251+
methodKind = methodDecl.ExplicitInterfaceImplementations.Select(m => m.MethodKind).First();
247252
}
248253

249254
switch (methodKind)

0 commit comments

Comments
 (0)