From 673b4466b45fad1431a68305dec24dec3cd32068 Mon Sep 17 00:00:00 2001 From: peter Date: Tue, 12 Jul 2011 19:52:17 +0200 Subject: [PATCH] remove debug output --- .../completion/JavaAutoPopupTest.groovy | 20 ------------------- .../lookup/impl/LookupActionHandler.java | 10 ---------- .../codeInsight/lookup/impl/LookupImpl.java | 10 ---------- .../lookup/impl/LookupManagerImpl.java | 10 ---------- 4 files changed, 50 deletions(-) diff --git a/java/java-tests/testSrc/com/intellij/codeInsight/completion/JavaAutoPopupTest.groovy b/java/java-tests/testSrc/com/intellij/codeInsight/completion/JavaAutoPopupTest.groovy index bae8e161e2da..d8474f80f82b 100644 --- a/java/java-tests/testSrc/com/intellij/codeInsight/completion/JavaAutoPopupTest.groovy +++ b/java/java-tests/testSrc/com/intellij/codeInsight/completion/JavaAutoPopupTest.groovy @@ -37,7 +37,6 @@ import com.intellij.openapi.extensions.LoadingOrder import com.intellij.openapi.fileEditor.FileEditorManager import com.intellij.openapi.progress.ProgressManager import com.intellij.psi.PsiFile -import com.intellij.codeInsight.lookup.impl.LookupActionHandler /** * @author peter @@ -554,15 +553,7 @@ public interface Test { } - @Override - protected void tearDown() { - LookupActionHandler.DEBUG = false - super.tearDown() - } - public void testLeftRightMovements() { - LookupActionHandler.DEBUG = true - myFixture.configureByText("a.java", """ class Foo { void foo(String iterable) { @@ -591,17 +582,6 @@ public interface Test { for (i in 0.."iter".size()) { edt { myFixture.performEditorAction(IdeActions.ACTION_EDITOR_MOVE_CARET_RIGHT) } - println myFixture.editor.caretModel.offset - println myFixture.editor.document.text[myFixture.editor.caretModel.offset] - } - if (lookup) { - println lookup.items - println myFixture.editor.document.text - println myFixture.editor.caretModel.offset - println myFixture.editor.settings.virtualSpace - println project - println LookupManager.getInstance(project) - println LookupManager.getInstance(project).getActiveLookup() } assert !lookup } diff --git a/platform/lang-impl/src/com/intellij/codeInsight/lookup/impl/LookupActionHandler.java b/platform/lang-impl/src/com/intellij/codeInsight/lookup/impl/LookupActionHandler.java index edbcc1856b45..82b735c466ea 100644 --- a/platform/lang-impl/src/com/intellij/codeInsight/lookup/impl/LookupActionHandler.java +++ b/platform/lang-impl/src/com/intellij/codeInsight/lookup/impl/LookupActionHandler.java @@ -36,7 +36,6 @@ import com.intellij.ui.ListScrollingUtil; * @author yole */ public abstract class LookupActionHandler extends EditorActionHandler { - public static boolean DEBUG = false; protected final EditorActionHandler myOriginalHandler; private final boolean myRequireFocusedLookup; @@ -204,21 +203,12 @@ public abstract class LookupActionHandler extends EditorActionHandler { char c = seq.charAt(offset); CharFilter.Result lookupAction = TypedHandler.getLookupAction(c, lookup); - if (DEBUG) { - System.out.println("LookupActionHandler$RightHandler.executeInLookup"); - System.out.println("c = '" + c + "'"); - System.out.println("lookupAction = " + lookupAction); - } if (lookupAction != CharFilter.Result.ADD_TO_PREFIX || Character.isWhitespace(c)) { myOriginalHandler.execute(editor, context); return; } - if (DEBUG) { - System.out.println("LookupActionHandler$RightHandler.executeInLookup2"); - } - lookup.performGuardedChange(new Runnable() { @Override public void run() { diff --git a/platform/lang-impl/src/com/intellij/codeInsight/lookup/impl/LookupImpl.java b/platform/lang-impl/src/com/intellij/codeInsight/lookup/impl/LookupImpl.java index b117f9f2b923..b6cfad896daf 100644 --- a/platform/lang-impl/src/com/intellij/codeInsight/lookup/impl/LookupImpl.java +++ b/platform/lang-impl/src/com/intellij/codeInsight/lookup/impl/LookupImpl.java @@ -850,11 +850,6 @@ public class LookupImpl extends LightweightHint implements LookupEx, Disposable final CaretListener caretListener = new CaretListener() { public void caretPositionChanged(CaretEvent e) { - if (LookupActionHandler.DEBUG) { - System.out.println("LookupImpl.caretPositionChanged"); - System.out.println("myChangeGuard = " + myChangeGuard); - System.out.println("myDisposed = " + myDisposed); - } if (!myChangeGuard) { hide(); } @@ -1186,11 +1181,6 @@ public class LookupImpl extends LightweightHint implements LookupEx, Disposable LOG.error("Disposing under a change guard"); } - if (LookupActionHandler.DEBUG) { - System.out.println("LookupImpl.doHide"); - System.out.println("fireCanceled = " + fireCanceled); - } - if (myDisposed) { LOG.error(disposeTrace); } diff --git a/platform/lang-impl/src/com/intellij/codeInsight/lookup/impl/LookupManagerImpl.java b/platform/lang-impl/src/com/intellij/codeInsight/lookup/impl/LookupManagerImpl.java index 070efd1f52e7..8c3571a758ea 100644 --- a/platform/lang-impl/src/com/intellij/codeInsight/lookup/impl/LookupManagerImpl.java +++ b/platform/lang-impl/src/com/intellij/codeInsight/lookup/impl/LookupManagerImpl.java @@ -188,10 +188,6 @@ public class LookupManagerImpl extends LookupManager { } private void lookupClosed() { - if (LookupActionHandler.DEBUG) { - System.out.println("LookupManagerImpl.lookupClosed"); - } - alarm.cancelAllRequests(); if (daemonCodeAnalyzer != null) { daemonCodeAnalyzer.setUpdateByTimerEnabled(previousUpdate); @@ -206,12 +202,6 @@ public class LookupManagerImpl extends LookupManager { myPropertyChangeSupport.firePropertyChange(PROP_ACTIVE_LOOKUP, lookup, null); Disposer.dispose(connector); - if (LookupActionHandler.DEBUG) { - System.out.println("LookupManagerImpl.lookupClosed2"); - System.out.println("myActiveLookup = " + myActiveLookup); - System.out.println("this = " + LookupManagerImpl.this); - System.out.println("project = " + myProject); - } } });