From 00ebd967e23b57781c658f35e60861ac1be4d9b9 Mon Sep 17 00:00:00 2001 From: Andrey Breslav Date: Tue, 22 Apr 2014 19:35:45 +0400 Subject: [PATCH] AnalyzerFacadeWithCache.analyzeFileWithCache() is inlined --- .../plugin/actions/CopyAsDiagnosticTestAction.java | 4 ++-- .../jet/plugin/actions/ShowExpressionTypeAction.java | 4 ++-- .../codeInsight/JetTypeDeclarationProvider.java | 4 ++-- .../jet/plugin/debugger/JetPositionManager.java | 4 ++-- .../jet/plugin/highlighter/DebugInfoAnnotator.java | 4 ++-- .../plugin/highlighter/DeclarationHintSupport.java | 4 ++-- .../jet/plugin/highlighter/JetLineMarkerProvider.java | 3 ++- .../jet/plugin/highlighter/JetPsiChecker.java | 4 ++-- .../MoveLambdaInsideParenthesesIntention.kt | 5 ++--- .../ReplaceWithInfixFunctionCallIntention.kt | 10 ++-------- .../intentions/SpecifyTypeExplicitlyAction.java | 6 +++--- .../internal/codewindow/BytecodeToolWindow.java | 5 ++--- .../internal/resolvewindow/ResolveToolWindow.java | 4 ++-- .../jet/plugin/project/AnalyzerFacadeWithCache.java | 11 ----------- .../jet/plugin/quickfix/AddFunctionParametersFix.java | 4 ++-- .../plugin/quickfix/AddFunctionToSupertypeFix.java | 2 +- .../jet/plugin/quickfix/AddNameToArgumentFix.java | 4 ++-- .../jet/plugin/quickfix/CastExpressionFix.java | 4 ++-- .../quickfix/ChangeFunctionLiteralReturnTypeFix.java | 4 ++-- .../quickfix/ChangeFunctionLiteralSignatureFix.java | 4 ++-- .../plugin/quickfix/ChangeFunctionReturnTypeFix.java | 8 ++++---- .../plugin/quickfix/ChangeFunctionSignatureFix.java | 8 ++++---- .../plugin/quickfix/ChangeVariableMutabilityFix.java | 4 ++-- .../jet/plugin/quickfix/ChangeVariableTypeFix.java | 6 +++--- .../plugin/quickfix/ChangeVisibilityModifierFix.java | 4 ++-- .../plugin/quickfix/CreateFunctionFromUsageFix.java | 6 +++--- .../quickfix/QuickFixFactoryForTypeMismatchError.java | 4 ++-- .../jetbrains/jet/plugin/quickfix/QuickFixUtil.java | 6 +++--- .../plugin/quickfix/RemoveFunctionParametersFix.java | 4 ++-- .../plugin/structureView/JetStructureViewElement.java | 4 ++-- .../jet/plugin/DirectiveBasedActionUtils.java | 4 ++-- .../codeInsight/AbstractInsertImportOnPasteTest.java | 4 ++-- .../jet/plugin/highlighter/NoErrorsInStdlibTest.kt | 6 +++--- .../plugin/refactoring/rename/AbstractRenameTest.kt | 4 ++-- 34 files changed, 74 insertions(+), 92 deletions(-) diff --git a/idea/src/org/jetbrains/jet/plugin/actions/CopyAsDiagnosticTestAction.java b/idea/src/org/jetbrains/jet/plugin/actions/CopyAsDiagnosticTestAction.java index 1f7e8ca5419..9933509aac9 100644 --- a/idea/src/org/jetbrains/jet/plugin/actions/CopyAsDiagnosticTestAction.java +++ b/idea/src/org/jetbrains/jet/plugin/actions/CopyAsDiagnosticTestAction.java @@ -27,7 +27,7 @@ import org.jetbrains.jet.checkers.CheckerTestUtil; import org.jetbrains.jet.lang.diagnostics.Diagnostic; import org.jetbrains.jet.lang.psi.JetFile; import org.jetbrains.jet.lang.resolve.BindingContext; -import org.jetbrains.jet.plugin.project.AnalyzerFacadeWithCache; +import org.jetbrains.jet.plugin.caches.resolve.ResolvePackage; import java.awt.*; import java.awt.datatransfer.Clipboard; @@ -43,7 +43,7 @@ public class CopyAsDiagnosticTestAction extends AnAction { PsiFile psiFile = e.getData(LangDataKeys.PSI_FILE); assert editor != null && psiFile != null; - BindingContext bindingContext = AnalyzerFacadeWithCache.analyzeFileWithCache((JetFile) psiFile).getBindingContext(); + BindingContext bindingContext = ResolvePackage.getAnalysisResults((JetFile) psiFile).getBindingContext(); List diagnostics = CheckerTestUtil.getDiagnosticsIncludingSyntaxErrors(bindingContext, psiFile); String result = CheckerTestUtil.addDiagnosticMarkersToText(psiFile, diagnostics).toString(); diff --git a/idea/src/org/jetbrains/jet/plugin/actions/ShowExpressionTypeAction.java b/idea/src/org/jetbrains/jet/plugin/actions/ShowExpressionTypeAction.java index 16c69eedeb5..a2570532c37 100644 --- a/idea/src/org/jetbrains/jet/plugin/actions/ShowExpressionTypeAction.java +++ b/idea/src/org/jetbrains/jet/plugin/actions/ShowExpressionTypeAction.java @@ -30,7 +30,7 @@ import org.jetbrains.jet.lang.psi.JetFile; import org.jetbrains.jet.lang.resolve.BindingContext; import org.jetbrains.jet.lang.types.JetType; import org.jetbrains.jet.plugin.JetLanguage; -import org.jetbrains.jet.plugin.project.AnalyzerFacadeWithCache; +import org.jetbrains.jet.plugin.caches.resolve.ResolvePackage; import org.jetbrains.jet.renderer.DescriptorRenderer; public class ShowExpressionTypeAction extends AnAction { @@ -40,7 +40,7 @@ public class ShowExpressionTypeAction extends AnAction { PsiFile psiFile = e.getData(LangDataKeys.PSI_FILE); assert editor != null && psiFile != null; JetExpression expression; - BindingContext bindingContext = AnalyzerFacadeWithCache.analyzeFileWithCache((JetFile) psiFile).getBindingContext(); + BindingContext bindingContext = ResolvePackage.getAnalysisResults((JetFile) psiFile).getBindingContext(); if (editor.getSelectionModel().hasSelection()) { int startOffset = editor.getSelectionModel().getSelectionStart(); int endOffset = editor.getSelectionModel().getSelectionEnd(); diff --git a/idea/src/org/jetbrains/jet/plugin/codeInsight/JetTypeDeclarationProvider.java b/idea/src/org/jetbrains/jet/plugin/codeInsight/JetTypeDeclarationProvider.java index 00ee27907ae..2615721dbc6 100644 --- a/idea/src/org/jetbrains/jet/plugin/codeInsight/JetTypeDeclarationProvider.java +++ b/idea/src/org/jetbrains/jet/plugin/codeInsight/JetTypeDeclarationProvider.java @@ -26,14 +26,14 @@ import org.jetbrains.jet.lang.psi.JetFile; import org.jetbrains.jet.lang.resolve.BindingContext; import org.jetbrains.jet.lang.resolve.BindingContextUtils; import org.jetbrains.jet.lang.types.JetType; -import org.jetbrains.jet.plugin.project.AnalyzerFacadeWithCache; +import org.jetbrains.jet.plugin.caches.resolve.ResolvePackage; public class JetTypeDeclarationProvider implements TypeDeclarationProvider { @Override public PsiElement[] getSymbolTypeDeclarations(PsiElement symbol) { if (symbol instanceof JetElement && symbol.getContainingFile() instanceof JetFile) { BindingContext bindingContext = - AnalyzerFacadeWithCache.analyzeFileWithCache((JetFile) symbol.getContainingFile()).getBindingContext(); + ResolvePackage.getAnalysisResults((JetFile) symbol.getContainingFile()).getBindingContext(); DeclarationDescriptor descriptor = bindingContext.get(BindingContext.DECLARATION_TO_DESCRIPTOR, symbol); if (descriptor instanceof CallableDescriptor) { JetType type = ((CallableDescriptor) descriptor).getReturnType(); diff --git a/idea/src/org/jetbrains/jet/plugin/debugger/JetPositionManager.java b/idea/src/org/jetbrains/jet/plugin/debugger/JetPositionManager.java index c63fcf735df..2177c505641 100644 --- a/idea/src/org/jetbrains/jet/plugin/debugger/JetPositionManager.java +++ b/idea/src/org/jetbrains/jet/plugin/debugger/JetPositionManager.java @@ -34,6 +34,7 @@ import com.sun.jdi.request.ClassPrepareRequest; import org.jetbrains.annotations.NotNull; import org.jetbrains.annotations.Nullable; import org.jetbrains.annotations.TestOnly; +import org.jetbrains.jet.plugin.caches.resolve.ResolvePackage; import org.jetbrains.org.objectweb.asm.Type; import org.jetbrains.jet.analyzer.AnalyzeExhaust; import org.jetbrains.jet.codegen.ClassBuilderMode; @@ -53,7 +54,6 @@ import org.jetbrains.jet.lang.resolve.java.JvmClassName; import org.jetbrains.jet.lang.resolve.name.FqName; import org.jetbrains.jet.lang.types.lang.InlineStrategy; import org.jetbrains.jet.lang.types.lang.InlineUtil; -import org.jetbrains.jet.plugin.project.AnalyzerFacadeWithCache; import org.jetbrains.jet.plugin.util.DebuggerUtils; import java.util.*; @@ -196,7 +196,7 @@ public class JetPositionManager implements PositionManager { value = CachedValuesManager.getManager(file.getProject()).createCachedValue(new CachedValueProvider() { @Override public Result compute() { - AnalyzeExhaust analyzeExhaust = AnalyzerFacadeWithCache.analyzeFileWithCache(file); + AnalyzeExhaust analyzeExhaust = ResolvePackage.getAnalysisResults(file); analyzeExhaust.throwIfError(); Collection packageFiles = JetFilesProvider.getInstance(file.getProject()).allPackageFiles(file); diff --git a/idea/src/org/jetbrains/jet/plugin/highlighter/DebugInfoAnnotator.java b/idea/src/org/jetbrains/jet/plugin/highlighter/DebugInfoAnnotator.java index 0a30231aca3..2e1d80bced7 100644 --- a/idea/src/org/jetbrains/jet/plugin/highlighter/DebugInfoAnnotator.java +++ b/idea/src/org/jetbrains/jet/plugin/highlighter/DebugInfoAnnotator.java @@ -27,7 +27,7 @@ import org.jetbrains.jet.lang.psi.JetFile; import org.jetbrains.jet.lang.psi.JetReferenceExpression; import org.jetbrains.jet.lang.resolve.BindingContext; import org.jetbrains.jet.plugin.JetPluginUtil; -import org.jetbrains.jet.plugin.project.AnalyzerFacadeWithCache; +import org.jetbrains.jet.plugin.caches.resolve.ResolvePackage; /** * Quick showing possible problems with Kotlin internals in IDEA with tooltips @@ -49,7 +49,7 @@ public class DebugInfoAnnotator implements Annotator { if (element instanceof JetFile) { JetFile file = (JetFile) element; try { - BindingContext bindingContext = AnalyzerFacadeWithCache.analyzeFileWithCache(file).getBindingContext(); + BindingContext bindingContext = ResolvePackage.getAnalysisResults(file).getBindingContext(); DebugInfoUtil.markDebugAnnotations(file, bindingContext, new DebugInfoUtil.DebugInfoReporter() { @Override public void reportElementWithErrorType(@NotNull JetReferenceExpression expression) { diff --git a/idea/src/org/jetbrains/jet/plugin/highlighter/DeclarationHintSupport.java b/idea/src/org/jetbrains/jet/plugin/highlighter/DeclarationHintSupport.java index 248fc88b3a9..3adc97da3de 100644 --- a/idea/src/org/jetbrains/jet/plugin/highlighter/DeclarationHintSupport.java +++ b/idea/src/org/jetbrains/jet/plugin/highlighter/DeclarationHintSupport.java @@ -44,7 +44,7 @@ import org.jetbrains.jet.lang.psi.JetNamedDeclaration; import org.jetbrains.jet.lang.resolve.BindingContext; import org.jetbrains.jet.plugin.JetLanguage; import org.jetbrains.jet.plugin.JetPluginUtil; -import org.jetbrains.jet.plugin.project.AnalyzerFacadeWithCache; +import org.jetbrains.jet.plugin.caches.resolve.ResolvePackage; import org.jetbrains.jet.plugin.util.LongRunningReadTask; import org.jetbrains.jet.renderer.DescriptorRenderer; @@ -195,7 +195,7 @@ public class DeclarationHintSupport extends AbstractProjectComponent { DeclarationDescriptor descriptor = null; try { BindingContext bindingContext = - AnalyzerFacadeWithCache.analyzeFileWithCache(jetFile).getBindingContext(); + ResolvePackage.getAnalysisResults(jetFile).getBindingContext(); descriptor = bindingContext.getDiagnostics().forElement(declaration).isEmpty() ? bindingContext.get(BindingContext.DECLARATION_TO_DESCRIPTOR, declaration) : null; diff --git a/idea/src/org/jetbrains/jet/plugin/highlighter/JetLineMarkerProvider.java b/idea/src/org/jetbrains/jet/plugin/highlighter/JetLineMarkerProvider.java index a11b311fdc4..5de8b0f0810 100644 --- a/idea/src/org/jetbrains/jet/plugin/highlighter/JetLineMarkerProvider.java +++ b/idea/src/org/jetbrains/jet/plugin/highlighter/JetLineMarkerProvider.java @@ -62,6 +62,7 @@ import org.jetbrains.jet.lang.resolve.OverrideResolver; import org.jetbrains.jet.lexer.JetTokens; import org.jetbrains.jet.plugin.JetBundle; import org.jetbrains.jet.plugin.JetPluginUtil; +import org.jetbrains.jet.plugin.caches.resolve.ResolvePackage; import org.jetbrains.jet.plugin.codeInsight.JetFunctionPsiElementCellRenderer; import org.jetbrains.jet.plugin.project.AnalyzerFacadeWithCache; import org.jetbrains.jet.plugin.search.ideaExtensions.KotlinDefinitionsSearcher; @@ -327,7 +328,7 @@ public class JetLineMarkerProvider implements LineMarkerProvider { JetFile file = (JetFile)element.getContainingFile(); if (file == null) return ""; - BindingContext bindingContext = AnalyzerFacadeWithCache.analyzeFileWithCache(file).getBindingContext(); + BindingContext bindingContext = ResolvePackage.getAnalysisResults(file).getBindingContext(); DeclarationDescriptor descriptor = bindingContext.get(BindingContext.DECLARATION_TO_DESCRIPTOR, element); if (!(descriptor instanceof CallableMemberDescriptor)) { diff --git a/idea/src/org/jetbrains/jet/plugin/highlighter/JetPsiChecker.java b/idea/src/org/jetbrains/jet/plugin/highlighter/JetPsiChecker.java index b3a9f347844..2b1ec07d2e1 100644 --- a/idea/src/org/jetbrains/jet/plugin/highlighter/JetPsiChecker.java +++ b/idea/src/org/jetbrains/jet/plugin/highlighter/JetPsiChecker.java @@ -44,7 +44,7 @@ import org.jetbrains.jet.lang.psi.JetFile; import org.jetbrains.jet.lang.psi.JetReferenceExpression; import org.jetbrains.jet.lang.resolve.BindingContext; import org.jetbrains.jet.plugin.JetPluginUtil; -import org.jetbrains.jet.plugin.project.AnalyzerFacadeWithCache; +import org.jetbrains.jet.plugin.caches.resolve.ResolvePackage; import org.jetbrains.jet.plugin.quickfix.JetIntentionActionsFactory; import org.jetbrains.jet.plugin.quickfix.QuickFixes; @@ -100,7 +100,7 @@ public class JetPsiChecker implements Annotator, HighlightRangeExtension { JetFile file = (JetFile) element.getContainingFile(); - AnalyzeExhaust analyzeExhaust = AnalyzerFacadeWithCache.analyzeFileWithCache(file); + AnalyzeExhaust analyzeExhaust = ResolvePackage.getAnalysisResults(file); if (analyzeExhaust.isError()) { HighlighterPackage.updateHighlightingResult(file, true); diff --git a/idea/src/org/jetbrains/jet/plugin/intentions/MoveLambdaInsideParenthesesIntention.kt b/idea/src/org/jetbrains/jet/plugin/intentions/MoveLambdaInsideParenthesesIntention.kt index 3c0f5772fb6..5ce9ae1fb0e 100644 --- a/idea/src/org/jetbrains/jet/plugin/intentions/MoveLambdaInsideParenthesesIntention.kt +++ b/idea/src/org/jetbrains/jet/plugin/intentions/MoveLambdaInsideParenthesesIntention.kt @@ -19,9 +19,8 @@ package org.jetbrains.jet.plugin.intentions import com.intellij.openapi.editor.Editor import org.jetbrains.jet.lang.psi.JetCallExpression import org.jetbrains.jet.lang.psi.JetPsiFactory -import org.jetbrains.jet.plugin.project.AnalyzerFacadeWithCache import org.jetbrains.jet.lang.resolve.BindingContext -import org.jetbrains.jet.lang.psi.JetFile +import org.jetbrains.jet.plugin.caches.resolve.getBindingContext public class MoveLambdaInsideParenthesesIntention : JetSelfTargetingIntention( "move.lambda.inside.parentheses", javaClass()) { @@ -44,7 +43,7 @@ public class MoveLambdaInsideParenthesesIntention : JetSelfTargetingIntention { @NotNull private JetAddFunctionToClassifierAction createAction(Project project, Editor editor, JetFile file) { - BindingContext bindingContext = ResolvePackage.getLazyResolveSession(file).getBindingContext(); + BindingContext bindingContext = ResolvePackage.getBindingContext(element); return new JetAddFunctionToClassifierAction(project, editor, bindingContext, functionsToAdd); } diff --git a/idea/src/org/jetbrains/jet/plugin/quickfix/AddNameToArgumentFix.java b/idea/src/org/jetbrains/jet/plugin/quickfix/AddNameToArgumentFix.java index 6f27d2f6b31..f273b19db46 100644 --- a/idea/src/org/jetbrains/jet/plugin/quickfix/AddNameToArgumentFix.java +++ b/idea/src/org/jetbrains/jet/plugin/quickfix/AddNameToArgumentFix.java @@ -40,7 +40,7 @@ import org.jetbrains.jet.lang.types.JetType; import org.jetbrains.jet.lang.types.checker.JetTypeChecker; import org.jetbrains.jet.plugin.JetBundle; import org.jetbrains.jet.plugin.JetIcons; -import org.jetbrains.jet.plugin.project.AnalyzerFacadeWithCache; +import org.jetbrains.jet.plugin.caches.resolve.ResolvePackage; import javax.swing.*; import java.util.Collections; @@ -65,7 +65,7 @@ public class AddNameToArgumentFix extends JetIntentionAction { JetExpression callee = callElement.getCalleeExpression(); if (!(callee instanceof JetReferenceExpression)) return Collections.emptyList(); - BindingContext context = AnalyzerFacadeWithCache.analyzeFileWithCache(argument.getContainingJetFile()).getBindingContext(); + BindingContext context = ResolvePackage.getAnalysisResults(argument.getContainingJetFile()).getBindingContext(); ResolvedCall resolvedCall = context.get(BindingContext.RESOLVED_CALL, (JetReferenceExpression) callee); if (resolvedCall == null) return Collections.emptyList(); diff --git a/idea/src/org/jetbrains/jet/plugin/quickfix/CastExpressionFix.java b/idea/src/org/jetbrains/jet/plugin/quickfix/CastExpressionFix.java index 1df32caecca..991bf0de415 100644 --- a/idea/src/org/jetbrains/jet/plugin/quickfix/CastExpressionFix.java +++ b/idea/src/org/jetbrains/jet/plugin/quickfix/CastExpressionFix.java @@ -31,7 +31,7 @@ import org.jetbrains.jet.lang.resolve.BindingContext; import org.jetbrains.jet.lang.types.JetType; import org.jetbrains.jet.lang.types.checker.JetTypeChecker; import org.jetbrains.jet.plugin.JetBundle; -import org.jetbrains.jet.plugin.project.AnalyzerFacadeWithCache; +import org.jetbrains.jet.plugin.caches.resolve.ResolvePackage; import org.jetbrains.jet.renderer.DescriptorRenderer; public class CastExpressionFix extends JetIntentionAction { @@ -59,7 +59,7 @@ public class CastExpressionFix extends JetIntentionAction { @Override public boolean isAvailable(@NotNull Project project, Editor editor, PsiFile file) { if (!super.isAvailable(project, editor, file)) return false; - BindingContext context = AnalyzerFacadeWithCache.analyzeFileWithCache((JetFile) file).getBindingContext(); + BindingContext context = ResolvePackage.getAnalysisResults((JetFile) file).getBindingContext(); JetType expressionType = context.get(BindingContext.EXPRESSION_TYPE, element); return expressionType != null && JetTypeChecker.INSTANCE.isSubtypeOf(type, expressionType); } diff --git a/idea/src/org/jetbrains/jet/plugin/quickfix/ChangeFunctionLiteralReturnTypeFix.java b/idea/src/org/jetbrains/jet/plugin/quickfix/ChangeFunctionLiteralReturnTypeFix.java index 4b684125d98..49d51029f4c 100644 --- a/idea/src/org/jetbrains/jet/plugin/quickfix/ChangeFunctionLiteralReturnTypeFix.java +++ b/idea/src/org/jetbrains/jet/plugin/quickfix/ChangeFunctionLiteralReturnTypeFix.java @@ -34,7 +34,7 @@ import org.jetbrains.jet.lang.types.TypeUtils; import org.jetbrains.jet.lang.types.checker.JetTypeChecker; import org.jetbrains.jet.lang.types.lang.KotlinBuiltIns; import org.jetbrains.jet.plugin.JetBundle; -import org.jetbrains.jet.plugin.project.AnalyzerFacadeWithCache; +import org.jetbrains.jet.plugin.caches.resolve.ResolvePackage; import org.jetbrains.jet.renderer.DescriptorRenderer; import java.util.LinkedList; @@ -50,7 +50,7 @@ public class ChangeFunctionLiteralReturnTypeFix extends JetIntentionAction @Override public IntentionAction createAction(Diagnostic diagnostic) { JetMultiDeclarationEntry entry = getMultiDeclarationEntryThatTypeMismatchComponentFunction(diagnostic); - BindingContext context = AnalyzerFacadeWithCache.analyzeFileWithCache((JetFile) entry.getContainingFile().getContainingFile()).getBindingContext(); + BindingContext context = ResolvePackage.getAnalysisResults((JetFile) entry.getContainingFile().getContainingFile()) + .getBindingContext(); ResolvedCall resolvedCall = context.get(BindingContext.COMPONENT_RESOLVED_CALL, entry); if (resolvedCall == null) return null; JetFunction componentFunction = (JetFunction) BindingContextUtils.descriptorToDeclaration(context, resolvedCall.getCandidateDescriptor()); @@ -164,7 +164,7 @@ public class ChangeFunctionReturnTypeFix extends JetIntentionAction public IntentionAction createAction(Diagnostic diagnostic) { JetExpression expression = QuickFixUtil.getParentElementOfType(diagnostic, JetExpression.class); assert expression != null : "HAS_NEXT_FUNCTION_TYPE_MISMATCH reported on element that is not within any expression"; - BindingContext context = AnalyzerFacadeWithCache.analyzeFileWithCache(expression.getContainingJetFile()).getBindingContext(); + BindingContext context = ResolvePackage.getAnalysisResults(expression.getContainingJetFile()).getBindingContext(); ResolvedCall resolvedCall = context.get(BindingContext.LOOP_RANGE_HAS_NEXT_RESOLVED_CALL, expression); if (resolvedCall == null) return null; JetFunction hasNextFunction = (JetFunction) BindingContextUtils.descriptorToDeclaration(context, resolvedCall.getCandidateDescriptor()); @@ -184,7 +184,7 @@ public class ChangeFunctionReturnTypeFix extends JetIntentionAction public IntentionAction createAction(Diagnostic diagnostic) { JetBinaryExpression expression = QuickFixUtil.getParentElementOfType(diagnostic, JetBinaryExpression.class); assert expression != null : "COMPARE_TO_TYPE_MISMATCH reported on element that is not within any expression"; - BindingContext context = AnalyzerFacadeWithCache.analyzeFileWithCache(expression.getContainingJetFile()).getBindingContext(); + BindingContext context = ResolvePackage.getAnalysisResults(expression.getContainingJetFile()).getBindingContext(); ResolvedCall resolvedCall = context.get(BindingContext.RESOLVED_CALL, expression.getOperationReference()); if (resolvedCall == null) return null; PsiElement compareTo = BindingContextUtils.descriptorToDeclaration(context, resolvedCall.getCandidateDescriptor()); diff --git a/idea/src/org/jetbrains/jet/plugin/quickfix/ChangeFunctionSignatureFix.java b/idea/src/org/jetbrains/jet/plugin/quickfix/ChangeFunctionSignatureFix.java index 13e810711e1..ff7760b48fa 100644 --- a/idea/src/org/jetbrains/jet/plugin/quickfix/ChangeFunctionSignatureFix.java +++ b/idea/src/org/jetbrains/jet/plugin/quickfix/ChangeFunctionSignatureFix.java @@ -38,7 +38,7 @@ import org.jetbrains.jet.lang.types.JetType; import org.jetbrains.jet.lang.types.checker.JetTypeChecker; import org.jetbrains.jet.lang.types.lang.KotlinBuiltIns; import org.jetbrains.jet.plugin.JetBundle; -import org.jetbrains.jet.plugin.project.AnalyzerFacadeWithCache; +import org.jetbrains.jet.plugin.caches.resolve.ResolvePackage; import org.jetbrains.jet.plugin.refactoring.JetNameSuggester; import org.jetbrains.jet.plugin.refactoring.JetNameValidator; import org.jetbrains.jet.plugin.refactoring.changeSignature.JetParameterInfo; @@ -77,7 +77,7 @@ public abstract class ChangeFunctionSignatureFix extends JetIntentionAction declarations = BindingContextUtils.callableDescriptorToDeclarations(bindingContext, functionDescriptor); if (declarations.isEmpty()) { return false; @@ -164,7 +164,7 @@ public abstract class ChangeFunctionSignatureFix extends JetIntentionAction>) diagnostic; JetFunctionLiteral functionLiteral = diagnosticWithParameters.getPsiElement(); BindingContext bindingContext = - AnalyzerFacadeWithCache.analyzeFileWithCache(functionLiteral.getContainingJetFile()).getBindingContext(); + ResolvePackage.getAnalysisResults(functionLiteral.getContainingJetFile()).getBindingContext(); DeclarationDescriptor descriptor = bindingContext.get(BindingContext.DECLARATION_TO_DESCRIPTOR, functionLiteral); if (descriptor instanceof FunctionDescriptor) { @@ -219,7 +219,7 @@ public abstract class ChangeFunctionSignatureFix extends JetIntentionAction resolvedCall = context.get(BindingContext.COMPONENT_RESOLVED_CALL, entry); if (resolvedCall == null) return null; JetFunction componentFunction = (JetFunction) BindingContextUtils.descriptorToDeclaration(context, resolvedCall.getCandidateDescriptor()); @@ -130,7 +130,7 @@ public class ChangeVariableTypeFix extends JetIntentionAction { @@ -71,7 +71,7 @@ public class ChangeVisibilityModifierFix extends JetIntentionAction resolvedCall = context.get(BindingContext.RESOLVED_CALL, callExpression.getCalleeExpression()); if (resolvedCall == null) return null; PsiElement declaration = BindingContextUtils.descriptorToDeclaration(context, resolvedCall.getCandidateDescriptor()); diff --git a/idea/src/org/jetbrains/jet/plugin/quickfix/RemoveFunctionParametersFix.java b/idea/src/org/jetbrains/jet/plugin/quickfix/RemoveFunctionParametersFix.java index f0e0f978aff..3508aec6f32 100644 --- a/idea/src/org/jetbrains/jet/plugin/quickfix/RemoveFunctionParametersFix.java +++ b/idea/src/org/jetbrains/jet/plugin/quickfix/RemoveFunctionParametersFix.java @@ -25,7 +25,7 @@ import org.jetbrains.jet.lang.descriptors.ValueParameterDescriptor; import org.jetbrains.jet.lang.psi.JetFile; import org.jetbrains.jet.lang.resolve.BindingContext; import org.jetbrains.jet.plugin.JetBundle; -import org.jetbrains.jet.plugin.project.AnalyzerFacadeWithCache; +import org.jetbrains.jet.plugin.caches.resolve.ResolvePackage; import org.jetbrains.jet.plugin.refactoring.changeSignature.JetChangeSignatureConfiguration; import org.jetbrains.jet.plugin.refactoring.changeSignature.JetChangeSignatureData; @@ -54,7 +54,7 @@ public class RemoveFunctionParametersFix extends ChangeFunctionSignatureFix { @Override protected void invoke(@NotNull Project project, Editor editor, JetFile file) { - BindingContext bindingContext = AnalyzerFacadeWithCache.analyzeFileWithCache(file).getBindingContext(); + BindingContext bindingContext = ResolvePackage.getAnalysisResults(file).getBindingContext(); runChangeSignature(project, functionDescriptor, new JetChangeSignatureConfiguration() { @Override public void configure( diff --git a/idea/src/org/jetbrains/jet/plugin/structureView/JetStructureViewElement.java b/idea/src/org/jetbrains/jet/plugin/structureView/JetStructureViewElement.java index 79194ee67af..67edff8f6ca 100644 --- a/idea/src/org/jetbrains/jet/plugin/structureView/JetStructureViewElement.java +++ b/idea/src/org/jetbrains/jet/plugin/structureView/JetStructureViewElement.java @@ -33,7 +33,7 @@ import org.jetbrains.jet.lang.psi.*; import org.jetbrains.jet.lang.resolve.BindingContext; import org.jetbrains.jet.lang.resolve.DescriptorUtils; import org.jetbrains.jet.lang.types.JetType; -import org.jetbrains.jet.plugin.project.AnalyzerFacadeWithCache; +import org.jetbrains.jet.plugin.caches.resolve.ResolvePackage; import org.jetbrains.jet.renderer.DescriptorRenderer; import javax.swing.*; @@ -119,7 +119,7 @@ public class JetStructureViewElement implements StructureViewTreeElement { public TreeElement[] getChildren() { if (myElement instanceof JetFile) { JetFile jetFile = (JetFile) myElement; - context = AnalyzerFacadeWithCache.analyzeFileWithCache(jetFile).getBindingContext(); + context = ResolvePackage.getAnalysisResults(jetFile).getBindingContext(); return wrapDeclarations(jetFile.getDeclarations()); } else if (myElement instanceof JetClass) { diff --git a/idea/tests/org/jetbrains/jet/plugin/DirectiveBasedActionUtils.java b/idea/tests/org/jetbrains/jet/plugin/DirectiveBasedActionUtils.java index 75ccf0a7d7f..7d97ed9df68 100644 --- a/idea/tests/org/jetbrains/jet/plugin/DirectiveBasedActionUtils.java +++ b/idea/tests/org/jetbrains/jet/plugin/DirectiveBasedActionUtils.java @@ -29,8 +29,8 @@ import org.jetbrains.jet.analyzer.AnalyzeExhaust; import org.jetbrains.jet.lang.diagnostics.Diagnostic; import org.jetbrains.jet.lang.diagnostics.Severity; import org.jetbrains.jet.lang.psi.JetFile; +import org.jetbrains.jet.plugin.caches.resolve.ResolvePackage; import org.jetbrains.jet.plugin.highlighter.IdeErrorMessages; -import org.jetbrains.jet.plugin.project.AnalyzerFacadeWithCache; import java.util.Collection; import java.util.Collections; @@ -45,7 +45,7 @@ public class DirectiveBasedActionUtils { return; } - AnalyzeExhaust exhaust = AnalyzerFacadeWithCache.analyzeFileWithCache(file); + AnalyzeExhaust exhaust = ResolvePackage.getAnalysisResults(file); Collection diagnostics = exhaust.getBindingContext().getDiagnostics().all(); Collection errorDiagnostics = Collections2.filter(diagnostics, new Predicate() { diff --git a/idea/tests/org/jetbrains/jet/plugin/codeInsight/AbstractInsertImportOnPasteTest.java b/idea/tests/org/jetbrains/jet/plugin/codeInsight/AbstractInsertImportOnPasteTest.java index 75b0a4c6094..d8597302605 100644 --- a/idea/tests/org/jetbrains/jet/plugin/codeInsight/AbstractInsertImportOnPasteTest.java +++ b/idea/tests/org/jetbrains/jet/plugin/codeInsight/AbstractInsertImportOnPasteTest.java @@ -32,7 +32,7 @@ import org.jetbrains.jet.lang.psi.JetFile; import org.jetbrains.jet.lang.psi.JetReferenceExpression; import org.jetbrains.jet.plugin.JetLightCodeInsightFixtureTestCase; import org.jetbrains.jet.plugin.PluginTestCaseBase; -import org.jetbrains.jet.plugin.project.AnalyzerFacadeWithCache; +import org.jetbrains.jet.plugin.caches.resolve.ResolvePackage; import java.io.File; import java.util.List; @@ -97,7 +97,7 @@ public abstract class AbstractInsertImportOnPasteTest extends JetLightCodeInsigh } private static void checkNoUnresolvedReferences(@NotNull final JetFile file) { - AnalyzeExhaust exhaust = AnalyzerFacadeWithCache.analyzeFileWithCache(file); + AnalyzeExhaust exhaust = ResolvePackage.getAnalysisResults(file); for (Diagnostic diagnostic : exhaust.getBindingContext().getDiagnostics()) { if (Errors.UNRESOLVED_REFERENCE_DIAGNOSTICS.contains(diagnostic.getFactory())) { List textRanges = diagnostic.getTextRanges(); diff --git a/idea/tests/org/jetbrains/jet/plugin/highlighter/NoErrorsInStdlibTest.kt b/idea/tests/org/jetbrains/jet/plugin/highlighter/NoErrorsInStdlibTest.kt index 9cb63ed05e2..d1aa1dd76db 100644 --- a/idea/tests/org/jetbrains/jet/plugin/highlighter/NoErrorsInStdlibTest.kt +++ b/idea/tests/org/jetbrains/jet/plugin/highlighter/NoErrorsInStdlibTest.kt @@ -23,14 +23,14 @@ import com.intellij.openapi.roots.ModuleRootManager import org.jetbrains.jet.plugin.JetJdkAndLibraryProjectDescriptor import com.intellij.openapi.roots.OrderRootType import com.intellij.openapi.vfs.VfsUtil -import org.jetbrains.jet.plugin.project.AnalyzerFacadeWithCache import org.jetbrains.jet.lang.psi.JetFile -import org.jetbrains.jet.lang.diagnostics.Severity import org.jetbrains.jet.cli.common.messages.AnalyzerWithCompilerReport import org.jetbrains.jet.cli.common.messages.MessageCollectorPlainTextToStream import com.intellij.openapi.projectRoots.Sdk import org.jetbrains.jet.plugin.PluginTestCaseBase import kotlin.test.assertEquals +import org.jetbrains.jet.plugin.caches.resolve.getBindingContext +import org.jetbrains.jet.lang.diagnostics.Severity public class NoErrorsInStdlibTest: LightCodeInsightFixtureTestCase() { public fun testNoErrors() { @@ -46,7 +46,7 @@ public class NoErrorsInStdlibTest: LightCodeInsightFixtureTestCase() { if (!file!!.isDirectory()) { val psiFile = psiManager.findFile(file) if (psiFile is JetFile) { - var bindingContext = AnalyzerFacadeWithCache.analyzeFileWithCache(psiFile).getBindingContext() + var bindingContext = psiFile.getBindingContext() val errors = bindingContext.getDiagnostics().all().filter { it.getSeverity() == Severity.ERROR } if (!errors.isEmpty()) { diff --git a/idea/tests/org/jetbrains/jet/plugin/refactoring/rename/AbstractRenameTest.kt b/idea/tests/org/jetbrains/jet/plugin/refactoring/rename/AbstractRenameTest.kt index a9675ae5579..e5f3bd37605 100644 --- a/idea/tests/org/jetbrains/jet/plugin/refactoring/rename/AbstractRenameTest.kt +++ b/idea/tests/org/jetbrains/jet/plugin/refactoring/rename/AbstractRenameTest.kt @@ -35,7 +35,6 @@ import com.intellij.psi.search.GlobalSearchScope import com.intellij.psi.JavaPsiFacade import com.intellij.openapi.project.Project import com.intellij.openapi.module.Module -import org.jetbrains.jet.plugin.project.AnalyzerFacadeWithCache import org.jetbrains.jet.lang.resolve.BindingContext import java.util.Collections import org.jetbrains.jet.lang.resolve.name.Name @@ -47,6 +46,7 @@ import org.jetbrains.jet.lang.resolve.name.isSubpackageOf import org.jetbrains.jet.getString import org.jetbrains.jet.getNullableString import org.jetbrains.jet.plugin.search.allScope +import org.jetbrains.jet.plugin.caches.resolve.getBindingContext private enum class RenameType { JAVA_CLASS @@ -187,7 +187,7 @@ public abstract class AbstractRenameTest : MultiFileTestCase() { val document = FileDocumentManager.getInstance()!!.getDocument(mainFile)!! val jetFile = PsiDocumentManager.getInstance(context.project).getPsiFile(document) as JetFile - val bindingContext = AnalyzerFacadeWithCache.analyzeFileWithCache(jetFile).getBindingContext() + val bindingContext = jetFile.getBindingContext() val classDescriptor = bindingContext.get(BindingContext.FQNAME_TO_CLASS_DESCRIPTOR, classFqName)!! val psiElement = BindingContextUtils.descriptorToDeclaration(bindingContext, findDescriptorToRename(classDescriptor))!!