Add debug logging for name resolution failures

Author: cirras

CHANGELOG.md

@@ -10,6 +10,7 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
 ### Added
 
 - Support for MSBuild item and item metadata expressions in project files.
+- Debug logging for name resolution failures (enabled with `sonar-scanner -X`).
 - `ExhaustiveEnumCase` analysis rule, which flags `case` statements that do not handle all values in an enumeration.
 - `IterationPastHighBound` analysis rule, which flags `for` loops that iterate past the end of the collection.
 - `ExplicitBitwiseNot` analysis rule, which flags potentially incorrect bitwise `not` operations.

delphi-frontend/src/main/java/au/com/integradev/delphi/symbol/resolve/NameResolver.java

@@ -217,6 +217,18 @@ private void checkAmbiguity() {
     }
   }
 
+  private void checkNameResolutionFailed() {
+    if (LOG.isDebugEnabled() && nameResolutionFailed()) {
+      Node location = Iterables.getLast(names).getLocation();
+      LOG.debug(
+          "Name resolution failed on symbol '{}' ({} line {}:{})",
+          location.getImage(),
+          location.getUnitName(),
+          location.getBeginLine(),
+          location.getBeginColumn());
+    }
+  }
+
   void updateType(Type type) {
     currentType = type;
     ScopedType scopedType = extractScopedType(currentType);
@@ -233,6 +245,7 @@ public boolean isExplicitInvocation() {
 
   public void addToSymbolTable() {
     addResolvedDeclaration();
+    checkNameResolutionFailed();
 
     for (int i = 0; i < resolvedDeclarations.size(); ++i) {
       NameOccurrenceImpl name = names.get(i);