Merged PR 454133: Add more visitor target into ReferencesVisitor

Add SimpleType, SingleVariableDeclaration, VariableDeclarationFragment and MethodInvocation into ReferencesVisitor

CHANGELOG.md

@@ -4,6 +4,9 @@ The format is based on [Keep a Changelog](http://keepachangelog.com/en/1.0.0/)
 and this project adheres to [Semantic Versioning](http://semver.org/spec/v2.0.0.html).
 
 ## 0.1.3
+### Added
+- Add SimpleType, SingleVariableDeclaration, VariableDeclarationFragment and MethodInvocation into ReferencesVisitor
+
 ### Fixed
 - Stop enlisting the vertices and edges if the element does not have hover information
 

com.microsoft.java.lsif.core/src/com/microsoft/java/lsif/core/internal/visitors/ReferencesVisitor.java

@@ -11,7 +11,11 @@
 import org.eclipse.core.runtime.CoreException;
 import org.eclipse.core.runtime.NullProgressMonitor;
 import org.eclipse.jdt.core.dom.MethodDeclaration;
+import org.eclipse.jdt.core.dom.MethodInvocation;
+import org.eclipse.jdt.core.dom.SimpleType;
+import org.eclipse.jdt.core.dom.SingleVariableDeclaration;
 import org.eclipse.jdt.core.dom.TypeDeclaration;
+import org.eclipse.jdt.core.dom.VariableDeclarationFragment;
 import org.eclipse.jdt.ls.core.internal.JDTUtils;
 import org.eclipse.lsp4j.Location;
 import org.eclipse.lsp4j.Position;
@@ -32,10 +36,37 @@ public class ReferencesVisitor extends ProtocolVisitor {
 	public ReferencesVisitor() {
 	}
 
+	@Override
+	public boolean visit(SimpleType type) {
+		emitReferences(type.getStartPosition(), type.getLength());
+		return super.visit(type);
+	}
+
+	@Override
+	public boolean visit(SingleVariableDeclaration node) {
+		emitReferences(node.getName().getStartPosition(), node.getName().getLength());
+		return super.visit(node);
+	}
+
+	@Override
+	public boolean visit(VariableDeclarationFragment node) {
+		emitReferences(node.getName().getStartPosition(), node.getName().getLength());
+		return super.visit(node);
+	}
+
+	@Override
+	public boolean visit(MethodInvocation node) {
+		emitReferences(node.getName().getStartPosition(), node.getName().getLength());
+		if (node.getExpression() != null) {
+			emitReferences(node.getExpression().getStartPosition(), node.getExpression().getLength());
+		}
+		return super.visit(node);
+	}
+
 	@Override
 	public boolean visit(MethodDeclaration node) {
 		emitReferences(node.getName().getStartPosition(), node.getName().getLength());
-		return false;
+		return super.visit(node);
 	}
 
 	@Override