diff --git a/idea/src/org/jetbrains/jet/plugin/libraries/JetSourceNavigationHelper.java b/idea/src/org/jetbrains/jet/plugin/libraries/JetSourceNavigationHelper.java index 58d26e31400..df460df4712 100644 --- a/idea/src/org/jetbrains/jet/plugin/libraries/JetSourceNavigationHelper.java +++ b/idea/src/org/jetbrains/jet/plugin/libraries/JetSourceNavigationHelper.java @@ -118,6 +118,7 @@ public class JetSourceNavigationHelper { return resultScope; } + @NotNull private static List findAllSourceFilesWhichContainIdentifier(@NotNull JetDeclaration jetDeclaration) { VirtualFile libraryFile = jetDeclaration.getContainingFile().getVirtualFile(); String name = jetDeclaration.getName(); @@ -139,6 +140,7 @@ public class JetSourceNavigationHelper { return jetFiles; } + @Nullable private static Pair getBindingContextAndMemberScopeForLibrarySources( @NotNull Decl decompiledDeclaration) { PsiElement declarationContainer = decompiledDeclaration.getParent(); @@ -187,8 +189,8 @@ public class JetSourceNavigationHelper { @Nullable private static JetDeclaration getSourcePropertyOrFunction( - final @NotNull Decl decompiledDeclaration, - NavigationStrategy navigationStrategy + @NotNull Decl decompiledDeclaration, + @NotNull NavigationStrategy navigationStrategy ) { String memberNameAsString = decompiledDeclaration.getName(); assert memberNameAsString != null; @@ -236,16 +238,16 @@ public class JetSourceNavigationHelper { } private interface NavigationStrategy { - boolean declarationAndDescriptorMatch(Decl declaration, Descr descriptor); + boolean declarationAndDescriptorMatch(@NotNull Decl declaration, @NotNull Descr descriptor); - Collection getCandidateDescriptors(JetScope scope, Name name); + @NotNull Collection getCandidateDescriptors(@NotNull JetScope scope, @NotNull Name name); @Nullable JetTypeReference getReceiverType(@NotNull Decl declaration); } private static class FunctionNavigationStrategy implements NavigationStrategy { @Override - public boolean declarationAndDescriptorMatch(JetFunction declaration, FunctionDescriptor descriptor) { + public boolean declarationAndDescriptorMatch(@NotNull JetFunction declaration, @NotNull FunctionDescriptor descriptor) { List declarationParameters = declaration.getValueParameters(); List descriptorParameters = descriptor.getValueParameters(); if (descriptorParameters.size() != declarationParameters.size()) { @@ -275,8 +277,9 @@ public class JetSourceNavigationHelper { return true; } + @NotNull @Override - public Collection getCandidateDescriptors(JetScope scope, Name name) { + public Collection getCandidateDescriptors(@NotNull JetScope scope, @NotNull Name name) { return scope.getFunctions(name); } @@ -289,12 +292,13 @@ public class JetSourceNavigationHelper { private static class PropertyNavigationStrategy implements NavigationStrategy { @Override - public boolean declarationAndDescriptorMatch(JetProperty declaration, VariableDescriptor descriptor) { + public boolean declarationAndDescriptorMatch(@NotNull JetProperty declaration, @NotNull VariableDescriptor descriptor) { return true; } + @NotNull @Override - public Collection getCandidateDescriptors(JetScope scope, Name name) { + public Collection getCandidateDescriptors(@NotNull JetScope scope, @NotNull Name name) { return scope.getProperties(name); }