cleanup

java/java-impl/src/com/intellij/codeInsight/daemon/impl/quickfix/ModifierFix.java

@@ -99,6 +99,7 @@ public class ModifierFix extends IntentionAndQuickFixAction {
     return myModifierList != null &&
            myModifierList.isValid() &&
            myModifierList.getManager().isInProject(myModifierList) &&
+           myModifierList.hasModifierProperty(myModifier) != myShouldHave &&
            (myVariable == null || myVariable.isValid());
   }
 

java/java-impl/src/com/intellij/codeInspection/dataFlow/ControlFlow.java

@@ -92,10 +92,6 @@ public class ControlFlow {
     myFields = fields;
   }
 
-  public void dump(PrintStream p) {
-    p.println(toString());
-  }
-
 
   public String toString() {
     StringBuilder result = new StringBuilder();

platform/lang-api/src/com/intellij/psi/codeStyle/CodeStyleSettings.java

@@ -164,7 +164,7 @@ public class CodeStyleSettings implements Cloneable, JDOMExternalizable {
       field.set(to, field.get(from));
     }
     else {
-      System.out.println("Field not copied " + field.getName());
+      throw new RuntimeException("Field not copied " + field.getName());
     }
   }
 

platform/platform-impl/src/com/intellij/openapi/command/impl/UndoManagerImpl.java

@@ -403,7 +403,7 @@ public class UndoManagerImpl extends UndoManager implements ProjectComponent, Ap
     return isUndoOrRedoAvailable(refs, false);
   }
 
-  private Collection<DocumentReference> getDocRefs(FileEditor editor) {
+  private static Collection<DocumentReference> getDocRefs(FileEditor editor) {
     if (editor instanceof TextEditor && ((TextEditor)editor).getEditor().isViewer()) return null;
     return getDocumentReferences(editor);
   }

platform/platform-impl/src/com/intellij/openapi/command/impl/UndoRedoStacksHolder.java

@@ -136,7 +136,7 @@ class UndoRedoStacksHolder {
     }
   }
 
-  private void doAddToStack(LinkedList<UndoableGroup> stack, UndoableGroup group, int limit) {
+  private static void doAddToStack(LinkedList<UndoableGroup> stack, UndoableGroup group, int limit) {
     if (!group.isUndoable() && stack.isEmpty()) return;
     
     stack.addLast(group);
@@ -187,7 +187,7 @@ class UndoRedoStacksHolder {
     }
   }
 
-  private void doInvalidateAllGlobalActions(LinkedList<UndoableGroup> stack) {
+  private static void doInvalidateAllGlobalActions(LinkedList<UndoableGroup> stack) {
     for (UndoableGroup g : stack) {
       g.invalidateIfGlobal();
     }