for -> foreach; generifying; other warning fixes
authorTagir Valeev <Tagir.Valeev@jetbrains.com>
Wed, 5 Sep 2018 13:14:55 +0000 (20:14 +0700)
committerTagir Valeev <Tagir.Valeev@jetbrains.com>
Wed, 5 Sep 2018 13:19:50 +0000 (20:19 +0700)
83 files changed:
java/compiler/forms-compiler/src/com/intellij/uiDesigner/compiler/AsmCodeGenerator.java
java/compiler/forms-compiler/src/com/intellij/uiDesigner/compiler/GridBagConverter.java
java/compiler/forms-compiler/src/com/intellij/uiDesigner/compiler/ListModelPropertyCodeGenerator.java
java/compiler/forms-compiler/src/com/intellij/uiDesigner/lw/CompiledClassPropertiesProvider.java
java/compiler/forms-compiler/src/com/intellij/uiDesigner/lw/FormLayoutSerializer.java
java/compiler/forms-compiler/src/com/intellij/uiDesigner/lw/LwButtonGroup.java
java/compiler/forms-compiler/src/com/intellij/uiDesigner/lw/LwContainer.java
java/compiler/forms-compiler/src/com/intellij/uiDesigner/lw/LwRootContainer.java
java/compiler/forms-compiler/src/com/intellij/uiDesigner/shared/BorderType.java
java/compiler/forms-compiler/testSrc/com/intellij/uiDesigner/core/GridBagConverterTest.java
java/compiler/javac2/src/com/intellij/ant/Javac2.java
java/compiler/javac2/src/com/intellij/ant/PrefixedPath.java
java/execution/impl/src/com/intellij/execution/MethodListDlg.java
java/java-analysis-impl/src/com/intellij/codeInspection/bytecodeAnalysis/asm/LiteAnalyzer.java
java/java-analysis-impl/src/com/intellij/codeInspection/bytecodeAnalysis/asm/LiteAnalyzerExt.java
java/java-impl/src/com/intellij/codeInspection/deadCode/DeadHTMLComposer.java
java/java-impl/src/com/intellij/cyclicDependencies/CyclicDependenciesBuilder.java
java/java-impl/src/com/intellij/ide/hierarchy/type/TypeHierarchyTreeStructure.java
java/java-impl/src/com/intellij/internal/diGraph/analyzer/GlobalAnalyzer.java
java/java-impl/src/com/intellij/refactoring/actions/EncapsulateFieldsAction.java
java/java-impl/src/com/intellij/refactoring/inheritanceToDelegation/InnerClassConstructor.java
java/java-impl/src/com/intellij/refactoring/migration/MigrationMap.java
java/java-impl/src/com/intellij/refactoring/migration/MigrationMapSet.java
java/java-impl/src/com/intellij/refactoring/util/classRefs/ClassReferenceScanner.java
java/java-tests/testSrc/com/intellij/java/refactoring/FixMethodJavadocTest.java
java/java-tests/testSrc/com/intellij/projectView/SameNamesJoiner.java
jps/antLayout/src/org/apache/tools/ant/taskdefs/PatchedJar.java
platform/lang-impl/src/com/intellij/ide/commander/ProjectListBuilder.java
platform/lang-impl/src/com/intellij/tools/FilterInfo.java
platform/platform-api/src/com/intellij/openapi/ui/MultiLineLabelUI.java
platform/platform-api/src/com/intellij/ui/ListUtil.java
platform/platform-api/src/com/intellij/ui/ReorderableListController.java
platform/platform-api/src/com/intellij/util/config/AbstractProperty.java
platform/platform-impl/src/com/intellij/openapi/diff/actions/PreviousDiffAction.java
platform/platform-impl/src/com/intellij/openapi/diff/impl/external/MultiLevelDiffTool.java
platform/platform-impl/src/com/intellij/openapi/keymap/impl/Converter01.java
platform/platform-impl/src/com/intellij/openapi/ui/playback/PathMacro.java
platform/platform-impl/src/com/intellij/openapi/vcs/changes/actions/migrate/MigrateToNewDiffUtil.java
platform/platform-impl/src/com/intellij/openapi/wm/impl/ColorThief.java
platform/platform-impl/src/com/intellij/openapi/wm/impl/WindowWatcher.java
platform/platform-impl/src/com/intellij/ui/HorizontalLabeledIcon.java
platform/platform-impl/src/com/intellij/ui/MultilineTreeCellRenderer.java
platform/platform-impl/src/com/intellij/ui/SelectionSaver.java
platform/platform-impl/src/net/sf/cglib/proxy/BridgeMethodGenerator.java
platform/platform-tests/testSrc/com/intellij/openapi/diff/impl/highlighting/MergeActionsTest.java
platform/platform-tests/testSrc/com/intellij/ui/TreeExpandCollapseTest.java
platform/platform-tests/testSrc/com/intellij/util/ui/TextToIconPaintTest.java
platform/util/src/com/intellij/openapi/diff/impl/fragments/FragmentListImpl.java
platform/util/src/com/intellij/openapi/diff/impl/highlighting/LineBlockDivider.java
platform/util/src/com/intellij/openapi/diff/impl/highlighting/List2D.java
platform/util/src/com/intellij/openapi/diff/impl/highlighting/Util.java
platform/util/src/com/intellij/openapi/diff/impl/processing/ByWord.java
platform/util/src/com/intellij/openapi/diff/impl/processing/DiffCorrection.java
platform/util/src/com/intellij/openapi/diff/impl/processing/UniteSameType.java
platform/util/src/com/intellij/util/ui/GeometryUtil.java
platform/util/src/com/intellij/util/ui/SplitBySeparatorPolicy.java
platform/util/testSrc/com/intellij/util/diff/UniqueLCSTest.java
platform/vcs-api/src/com/intellij/openapi/vcs/ui/ReplaceFileConfirmationDialog.java
platform/vcs-impl/src/com/intellij/ide/todo/CustomChangelistTodosTreeBuilder.java
platform/vcs-impl/src/com/intellij/openapi/diff/impl/patch/apply/GenericPatchApplier.java
platform/vcs-tests/testSrc/com/intellij/openapi/vcs/RangeBuilderTest.java
plugins/ant/src/com/intellij/lang/ant/config/impl/configuration/AntClasspathEditorPanel.java
plugins/cvs/cvs-plugin/src/com/intellij/cvsSupport2/actions/AbstractWatchAction.java
plugins/cvs/cvs-plugin/src/com/intellij/cvsSupport2/actions/ActionOnSelectedElement.java
plugins/cvs/cvs-plugin/src/com/intellij/cvsSupport2/cvshandlers/SelectedFiles.java
plugins/cvs/cvs-plugin/src/com/intellij/cvsSupport2/cvsoperations/cvsWatch/WatchersOperation.java
plugins/cvs/cvs-plugin/src/com/intellij/cvsSupport2/cvsoperations/javacvsSpecificImpls/AdminReaderForLightFiles.java
plugins/cvs/javacvs-src/org/netbeans/lib/cvsclient/RequestProcessor.java
plugins/cvs/javacvs-src/org/netbeans/lib/cvsclient/command/CvsFiles.java
plugins/cvs/javacvs-src/org/netbeans/lib/cvsclient/command/DirectoryPruner.java
plugins/cvs/javacvs-src/org/netbeans/lib/cvsclient/command/log/LogInformation.java
plugins/cvs/javacvs-src/org/netbeans/lib/cvsclient/command/status/StatusInformation.java
plugins/cvs/javacvs-src/org/netbeans/lib/cvsclient/command/watch/WatchCommand.java
plugins/cvs/javacvs-src/org/netbeans/lib/cvsclient/event/EventManager.java
plugins/cvs/smartcvs-src/org/netbeans/lib/cvsclient/admin/AdminWriter.java
plugins/cvs/smartcvs-src/org/netbeans/lib/cvsclient/file/LocalFileReader.java
plugins/cvs/smartcvs-src/org/netbeans/lib/cvsclient/util/DefaultIgnoreFileFilter.java
plugins/groovy/rt/src/org/jetbrains/plugins/groovy/grape/GrapeRunner.java
plugins/groovy/src/org/jetbrains/plugins/groovy/griffon/GriffonSourceInspector.java
plugins/groovy/src/org/jetbrains/plugins/groovy/refactoring/classMembers/GrMemberInfoStorage.java
plugins/hg4idea/testSrc/org/zmlx/hg4idea/test/HgUpdateTest.java
plugins/java-decompiler/engine/src/org/jetbrains/java/decompiler/modules/decompiler/exps/FunctionExprent.java
plugins/ui-designer/src/com/intellij/uiDesigner/wizard/Generator.java

index d2eed2d1ad3ec4d6f9aa9b28563bb982ffc4b32a..ebccd35fd689408597600e4a2c0f4b631dee0b79 100644 (file)
@@ -478,7 +478,7 @@ public class AsmCodeGenerator {
       LwContainer parent = container;
       while(parent != null) {
         final String layoutManager = parent.getLayoutManager();
-        if (layoutManager != null && layoutManager.length() > 0) {
+        if (layoutManager != null && !layoutManager.isEmpty()) {
           if (layoutManager.equals(UIFormXmlConstants.LAYOUT_FORM) &&
               !myContainerLayoutCodeGenerators.containsKey(UIFormXmlConstants.LAYOUT_FORM)) {
             myContainerLayoutCodeGenerators.put(UIFormXmlConstants.LAYOUT_FORM, new FormLayoutCodeGenerator());
@@ -499,8 +499,7 @@ public class AsmCodeGenerator {
                                              final int componentLocal) throws CodeGenerationException {
       // introspected properties
       final LwIntrospectedProperty[] introspectedProperties = lwComponent.getAssignedIntrospectedProperties();
-      for (int i = 0; i < introspectedProperties.length; i++) {
-        final LwIntrospectedProperty property = introspectedProperties[i];
+      for (final LwIntrospectedProperty property : introspectedProperties) {
         if (property instanceof LwIntroComponentProperty) {
           continue;
         }
@@ -534,7 +533,7 @@ public class AsmCodeGenerator {
               descriptor = "(C)V";
             }
             else {
-              descriptor = "(L"+Class.forName(propertyClass).getName().replace('.', '/') + ";)V";
+              descriptor = "(L" + Class.forName(propertyClass).getName().replace('.', '/') + ";)V";
             }
             final InstrumentationClassFinder.PseudoMethod setter = componentClass.findMethodInHierarchy(property.getWriteMethodName(),
                                                                                                         descriptor);
@@ -566,35 +565,35 @@ public class AsmCodeGenerator {
         Object value = lwComponent.getPropertyValue(property);
         Type setterArgType;
         if (propertyClass.equals(Integer.class.getName())) {
-          generator.push(((Integer) value).intValue());
+          generator.push(((Integer)value).intValue());
           setterArgType = Type.INT_TYPE;
         }
         else if (propertyClass.equals(Boolean.class.getName())) {
-          generator.push(((Boolean) value).booleanValue());
+          generator.push(((Boolean)value).booleanValue());
           setterArgType = Type.BOOLEAN_TYPE;
         }
         else if (propertyClass.equals(Double.class.getName())) {
-          generator.push(((Double) value).doubleValue());
+          generator.push(((Double)value).doubleValue());
           setterArgType = Type.DOUBLE_TYPE;
         }
         else if (propertyClass.equals(Float.class.getName())) {
-          generator.push(((Float) value).floatValue());
+          generator.push(((Float)value).floatValue());
           setterArgType = Type.FLOAT_TYPE;
         }
         else if (propertyClass.equals(Long.class.getName())) {
-          generator.push(((Long) value).longValue());
+          generator.push(((Long)value).longValue());
           setterArgType = Type.LONG_TYPE;
         }
         else if (propertyClass.equals(Short.class.getName())) {
-          generator.push(((Short) value).intValue());
+          generator.push(((Short)value).intValue());
           setterArgType = Type.SHORT_TYPE;
         }
         else if (propertyClass.equals(Byte.class.getName())) {
-          generator.push(((Byte) value).intValue());
+          generator.push(((Byte)value).intValue());
           setterArgType = Type.BYTE_TYPE;
         }
         else if (propertyClass.equals(Character.class.getName())) {
-          generator.push(((Character) value).charValue());
+          generator.push(((Character)value).charValue());
           setterArgType = Type.CHAR_TYPE;
         }
         else {
@@ -609,7 +608,7 @@ public class AsmCodeGenerator {
                              ? typeFromClassName(property.getDeclaringClassName())
                              : Type.getType(componentClass.getDescriptor());
         generator.invokeVirtual(declaringType, new Method(property.getWriteMethodName(),
-                                                          Type.VOID_TYPE, new Type[] { setterArgType } ));
+                                                          Type.VOID_TYPE, new Type[]{setterArgType}));
       }
 
       generateClientProperties(lwComponent, componentClass, generator, componentLocal);
@@ -671,11 +670,10 @@ public class AsmCodeGenerator {
       InstrumentationClassFinder.PseudoClass componentClass = getComponentClass(layoutCodeGenerator.mapComponentClass(component.getComponentClassName()), myFinder);
 
       final LwIntrospectedProperty[] introspectedProperties = component.getAssignedIntrospectedProperties();
-      for (int i = 0; i < introspectedProperties.length; i++) {
-        final LwIntrospectedProperty property = introspectedProperties[i];
+      for (final LwIntrospectedProperty property : introspectedProperties) {
         if (property instanceof LwIntroComponentProperty) {
-          String targetId = (String) component.getPropertyValue(property);
-          if (targetId != null && targetId.length() > 0) {
+          String targetId = (String)component.getPropertyValue(property);
+          if (targetId != null && !targetId.isEmpty()) {
             // we may have a reference property pointing to a component which is
             // no longer valid
             final Integer targetLocalInt = myIdToLocalMap.get(targetId);
@@ -688,7 +686,7 @@ public class AsmCodeGenerator {
                                    : Type.getType(componentClass.getDescriptor());
               generator.invokeVirtual(declaringType,
                                       new Method(property.getWriteMethodName(),
-                                                 Type.VOID_TYPE, new Type[] { typeFromClassName(property.getPropertyClassName()) } ));
+                                                 Type.VOID_TYPE, new Type[]{typeFromClassName(property.getPropertyClassName())}));
             }
           }
         }
@@ -709,8 +707,8 @@ public class AsmCodeGenerator {
         try {
           InstrumentationClassFinder.PseudoClass buttonGroupClass = null; // cached
           int groupLocal = generator.newLocal(ourButtonGroupType);
-          for(int groupIndex=0; groupIndex<groups.length; groupIndex++) {
-            String[] ids = groups [groupIndex].getComponentIds();
+          for (IButtonGroup group : groups) {
+            String[] ids = group.getComponentIds();
 
             if (ids.length > 0) {
               generator.newInstance(ourButtonGroupType);
@@ -718,18 +716,18 @@ public class AsmCodeGenerator {
               generator.invokeConstructor(ourButtonGroupType, Method.getMethod("void <init>()"));
               generator.storeLocal(groupLocal);
 
-              if (groups [groupIndex].isBound() && !myIgnoreCustomCreation) {
+              if (group.isBound() && !myIgnoreCustomCreation) {
                 if (buttonGroupClass == null) {
                   buttonGroupClass = myFinder.loadClass(ButtonGroup.class.getName());
                 }
-                validateFieldClass(groups [groupIndex].getName(), buttonGroupClass, null);
+                validateFieldClass(group.getName(), buttonGroupClass, null);
                 generator.loadThis();
                 generator.loadLocal(groupLocal);
-                generator.putField(getMainClassType(), groups [groupIndex].getName(), ourButtonGroupType);
+                generator.putField(getMainClassType(), group.getName(), ourButtonGroupType);
               }
 
-              for(int i = 0; i<ids.length; i++) {
-                Integer localInt = myIdToLocalMap.get(ids [i]);
+              for (String id : ids) {
+                Integer localInt = myIdToLocalMap.get(id);
                 if (localInt != null) {
                   generator.loadLocal(groupLocal);
                   generator.loadLocal(localInt.intValue());
@@ -793,7 +791,7 @@ public class AsmCodeGenerator {
         throw new CodeGenerationException(componentId, "Cannot bind: field does not exist: " + myClassToBind + "." + binding);
       }
 
-      final Type fieldType = Type.getType((String)myFieldDescMap.get(binding));
+      final Type fieldType = Type.getType(myFieldDescMap.get(binding));
       if (fieldType.getSort() != Type.OBJECT) {
         throw new CodeGenerationException(componentId, "Cannot bind: field is of primitive type: " + myClassToBind + "." + binding);
       }
@@ -862,7 +860,7 @@ public class AsmCodeGenerator {
           borderFactoryValue = StringDescriptor.create("com.intellij.ui.IdeBorderFactory$PlainSmallWithIndent");
           container.getDelegeeClientProperties().put(ourBorderFactoryClientProperty, borderFactoryValue);
         }
-        if (borderFactoryValue != null && borderFactoryValue.getValue().length() != 0) {
+        if (borderFactoryValue != null && !borderFactoryValue.getValue().isEmpty()) {
           borderFactoryType = typeFromClassName(borderFactoryValue.getValue());
         }
 
index 3c79f57d9a7b0369f4c8b3643179710dde554f54..3dfaf4408d746100a248171953df3d68bd56fbaf 100644 (file)
@@ -24,7 +24,6 @@ import com.intellij.uiDesigner.lw.*;
 import javax.swing.*;
 import java.awt.*;
 import java.util.ArrayList;
-import java.util.Iterator;
 import java.util.Map;
 
 /**
@@ -36,8 +35,8 @@ public class GridBagConverter {
   private int myVGap;
   private boolean mySameSizeHorz;
   private boolean mySameSizeVert;
-  private final ArrayList myComponents = new ArrayList();
-  private final ArrayList myConstraints = new ArrayList();
+  private final ArrayList<JComponent> myComponents = new ArrayList<JComponent>();
+  private final ArrayList<GridConstraints> myConstraints = new ArrayList<GridConstraints>();
   private int myLastRow = -1;
   private int myLastCol = -1;
 
@@ -79,10 +78,10 @@ public class GridBagConverter {
     }
     Result[] results = converter.convert();
     int componentIndex = 0;
-    for(int i=0; i<results.length; i++) {
-      if (!results [i].isFillerPanel) {
+    for (Result result : results) {
+      if (!result.isFillerPanel) {
         final LwComponent component = (LwComponent)container.getComponent(componentIndex++);
-        idToConstraintsMap.put(component.getId(), results [i]);
+        idToConstraintsMap.put(component.getId(), result);
       }
       // else generateFillerPanel(generator, componentLocal, results [i]);
     }
@@ -118,12 +117,12 @@ public class GridBagConverter {
   }
 
   public Result[] convert() {
-    ArrayList results = new ArrayList();
+    ArrayList<Result> results = new ArrayList<Result>();
     for(int i=0; i<myComponents.size(); i++) {
-      results.add(convert((JComponent) myComponents.get(i), (GridConstraints) myConstraints.get(i)));
+      results.add(convert(myComponents.get(i), myConstraints.get(i)));
     }
     //addFillerPanels(results);
-    final Result[] resultArray = (Result[])results.toArray(new Result[0]);
+    final Result[] resultArray = results.toArray(new Result[0]);
     if (myHGap > 0 || myVGap > 0) {
       applyGaps(resultArray);
     }
@@ -142,18 +141,17 @@ public class GridBagConverter {
     int rightGap = myHGap - myHGap/2;
     int topGap = myVGap / 2;
     int bottomGap = myVGap - myVGap/2;
-    for(int i=0; i<resultArray.length; i++) {
-      Result result = resultArray [i];
+    for (Result result : resultArray) {
       if (result.constraints.gridx > 0) {
         result.constraints.insets.left += leftGap;
       }
-      if (result.constraints.gridx + result.constraints.gridwidth-1 < myLastCol) {
+      if (result.constraints.gridx + result.constraints.gridwidth - 1 < myLastCol) {
         result.constraints.insets.right += rightGap;
       }
       if (result.constraints.gridy > 0) {
         result.constraints.insets.top += topGap;
       }
-      if (result.constraints.gridy + result.constraints.gridheight-1 < myLastRow) {
+      if (result.constraints.gridy + result.constraints.gridheight - 1 < myLastRow) {
         result.constraints.insets.bottom += bottomGap;
       }
     }
@@ -162,8 +160,7 @@ public class GridBagConverter {
   private static void makeSameSizes(final Result[] resultArray, boolean horizontal) {
     int minimum = -1;
     int preferred = -1;
-    for(int i=0; i<resultArray.length; i++) {
-      Result result = resultArray [i];
+    for (Result result : resultArray) {
       Dimension minSize = result.minimumSize != null || result.component == null
                           ? result.minimumSize
                           : result.component.getMinimumSize();
@@ -179,9 +176,7 @@ public class GridBagConverter {
     }
 
     if (minimum >= 0 || preferred >= 0) {
-      for(int i=0; i<resultArray.length; i++) {
-        Result result = resultArray [i];
-
+      for (Result result : resultArray) {
         if ((result.minimumSize != null || result.component != null) && minimum >= 0) {
           if (result.minimumSize == null) {
             result.minimumSize = result.component.getMinimumSize();
@@ -295,8 +290,8 @@ public class GridBagConverter {
       return 1.0;
     }
     boolean canGrow = ((policy & GridConstraints.SIZEPOLICY_CAN_GROW) != 0);
-    for (Iterator iterator = myConstraints.iterator(); iterator.hasNext();) {
-      GridConstraints otherConstraints = (GridConstraints)iterator.next();
+    for (Object myConstraint : myConstraints) {
+      GridConstraints otherConstraints = (GridConstraints)myConstraint;
 
       if (!constraintsIntersect(horizontal, constraints, otherConstraints)) {
         int otherPolicy = horizontal ? otherConstraints.getHSizePolicy() : otherConstraints.getVSizePolicy();
@@ -311,9 +306,9 @@ public class GridBagConverter {
     return 1.0;
   }
 
-  private boolean constraintsIntersect(final boolean horizontal,
-                                       final GridConstraints constraints,
-                                       final GridConstraints otherConstraints) {
+  private static boolean constraintsIntersect(final boolean horizontal,
+                                              final GridConstraints constraints,
+                                              final GridConstraints otherConstraints) {
     int start = constraints.getCell(!horizontal);
     int end = start + constraints.getSpan(!horizontal) - 1;
     int otherStart = otherConstraints.getCell(!horizontal);
index 10e1ce03e662801349605ab04b99aa4db7aef5f7..56939d4947acc79aa6bf91c1bfe80b19b6f3fe4b 100644 (file)
@@ -42,9 +42,9 @@ public class ListModelPropertyCodeGenerator extends PropertyCodeGenerator {
     generator.invokeConstructor(myListModelType, ourInitMethod);
     generator.storeLocal(listModelLocal);
 
-    for(int i=0; i<items.length; i++) {
+    for (String item : items) {
       generator.loadLocal(listModelLocal);
-      generator.push(items [i]);
+      generator.push(item);
       generator.invokeVirtual(myListModelType, ourAddElementMethod);
     }
 
index 51160d2ba13b052e257875bce352405910ea2f91..63b00153e5221d71bf0d61717dda794c8a01eb7f 100644 (file)
@@ -24,6 +24,7 @@ import java.beans.Introspector;
 import java.beans.PropertyDescriptor;
 import java.lang.reflect.Method;
 import java.util.HashMap;
+import java.util.Map;
 
 /**
  * @author Anton Katilin
@@ -31,14 +32,14 @@ import java.util.HashMap;
  */
 public final class CompiledClassPropertiesProvider implements PropertiesProvider {
   private final ClassLoader myLoader;
-  private final HashMap myCache;
+  private final HashMap<String, Map<String, LwIntrospectedProperty>> myCache;
 
   public CompiledClassPropertiesProvider(final ClassLoader loader) {
     if (loader == null) {
       throw new IllegalArgumentException("loader cannot be null");
     }
     myLoader = loader;
-    myCache = new HashMap();
+    myCache = new HashMap<String, Map<String, LwIntrospectedProperty>>();
   }
 
   @Override
@@ -67,11 +68,9 @@ public final class CompiledClassPropertiesProvider implements PropertiesProvider
       return null;
     }
 
-    final HashMap result = new HashMap();
+    final HashMap<String, LwIntrospectedProperty> result = new HashMap<String, LwIntrospectedProperty>();
     final PropertyDescriptor[] descriptors = beanInfo.getPropertyDescriptors();
-    for (int i = 0; i < descriptors.length; i++) {
-      final PropertyDescriptor descriptor = descriptors[i];
-
+    for (final PropertyDescriptor descriptor : descriptors) {
       final Method readMethod = descriptor.getReadMethod();
       final Method writeMethod = descriptor.getWriteMethod();
       final Class propertyType = descriptor.getPropertyType();
index 15275cbc01db24bd0fac6e1657a035e3d9714fd1..68c982242174cacf69ce3747f4aa182974296d41 100644 (file)
@@ -24,7 +24,6 @@ import com.jgoodies.forms.layout.FormLayout;
 import com.jgoodies.forms.layout.RowSpec;
 import org.jdom.Element;
 
-import java.util.Iterator;
 import java.util.List;
 
 /**
@@ -36,26 +35,24 @@ public class FormLayoutSerializer extends GridLayoutSerializer {
 
   public static FormLayoutSerializer INSTANCE = new FormLayoutSerializer();
 
-  public static CellConstraints.Alignment[] ourHorizontalAlignments = {
+  public static final CellConstraints.Alignment[] ourHorizontalAlignments = {
     CellConstraints.LEFT, CellConstraints.CENTER, CellConstraints.RIGHT, CellConstraints.FILL
   };
-  public static CellConstraints.Alignment[] ourVerticalAlignments = {
+  public static final CellConstraints.Alignment[] ourVerticalAlignments = {
     CellConstraints.TOP, CellConstraints.CENTER, CellConstraints.BOTTOM, CellConstraints.FILL
   };
 
   @Override
   void readLayout(Element element, LwContainer container) {
     FormLayout layout = new FormLayout();
-    final List rowSpecs = element.getChildren(UIFormXmlConstants.ELEMENT_ROWSPEC, element.getNamespace());
-    for (Iterator iterator = rowSpecs.iterator(); iterator.hasNext();) {
-      Element rowSpecElement = (Element) iterator.next();
+    final List<Element> rowSpecs = element.getChildren(UIFormXmlConstants.ELEMENT_ROWSPEC, element.getNamespace());
+    for (Element rowSpecElement : rowSpecs) {
       final String spec = LwXmlReader.getRequiredString(rowSpecElement, UIFormXmlConstants.ATTRIBUTE_VALUE);
       layout.appendRow(new RowSpec(spec));
     }
 
-    final List colSpecs = element.getChildren(UIFormXmlConstants.ELEMENT_COLSPEC, element.getNamespace());
-    for (Iterator iterator = colSpecs.iterator(); iterator.hasNext();) {
-      Element colSpecElement = (Element) iterator.next();
+    final List<Element> colSpecs = element.getChildren(UIFormXmlConstants.ELEMENT_COLSPEC, element.getNamespace());
+    for (Element colSpecElement : colSpecs) {
       final String spec = LwXmlReader.getRequiredString(colSpecElement, UIFormXmlConstants.ATTRIBUTE_VALUE);
       layout.appendColumn(new ColumnSpec(spec));
     }
@@ -73,7 +70,7 @@ public class FormLayoutSerializer extends GridLayoutSerializer {
 
   private static int[][] readGroups(final Element element, final String elementName) {
     final List groupElements = element.getChildren(elementName, element.getNamespace());
-    if (groupElements.size() == 0) return null;
+    if (groupElements.isEmpty()) return null;
     int[][] groups = new int[groupElements.size()][];
     for(int i=0; i<groupElements.size(); i++) {
       Element groupElement = (Element) groupElements.get(i);
index aaf6c87c803266075ec26852249f5b4b929c76e0..69b6d038ab6b6a4fbf8c4c144f31492f78a3b68d 100644 (file)
@@ -26,14 +26,13 @@ import java.util.Iterator;
  */
 public class LwButtonGroup implements IButtonGroup {
   private String myName;
-  private final ArrayList myComponentIds = new ArrayList();
+  private final ArrayList<String> myComponentIds = new ArrayList<String>();
   private boolean myBound;
 
   public void read(final Element element) {
     myName = element.getAttributeValue(UIFormXmlConstants.ATTRIBUTE_NAME);
     myBound = LwXmlReader.getOptionalBoolean(element, UIFormXmlConstants.ATTRIBUTE_BOUND, false);
-    for(Iterator i=element.getChildren().iterator(); i.hasNext();){
-      final Element child = (Element)i.next();
+    for (final Element child : element.getChildren()) {
       myComponentIds.add(child.getAttributeValue(UIFormXmlConstants.ATTRIBUTE_ID));
     }
   }
@@ -45,7 +44,7 @@ public class LwButtonGroup implements IButtonGroup {
 
   @Override
   public String[] getComponentIds() {
-    return (String[])myComponentIds.toArray(new String[0]);
+    return myComponentIds.toArray(new String[0]);
   }
 
   @Override
index 39db1d339c9ef01b03b00fbb6887ff1f1de9ca20..d374b9ae1e795295433c589f6528110bdc139d87 100644 (file)
@@ -261,8 +261,7 @@ public class LwContainer extends LwComponent implements IContainer{
    */
   protected final void readChildren(final Element element, final PropertiesProvider provider) throws Exception{
     final Element childrenElement = LwXmlReader.getRequiredChild(element, "children");
-    for(Iterator i=childrenElement.getChildren().iterator(); i.hasNext();){
-      final Element child = (Element)i.next();
+    for (final Element child : childrenElement.getChildren()) {
       final LwComponent component = createComponentFromTag(child);
       addComponent(component);
       component.read(child, provider);
index b2c26c19b4203d8049c58a6cd747701e30e94609..fe08db3e0c8c6ab15ac0bb8f078548930c069471 100644 (file)
@@ -22,7 +22,6 @@ import com.intellij.uiDesigner.compiler.Utils;
 import org.jdom.Element;
 
 import java.util.ArrayList;
-import java.util.Iterator;
 
 
 /**
@@ -32,10 +31,10 @@ import java.util.Iterator;
 public final class LwRootContainer extends LwContainer implements IRootContainer{
   private String myClassToBind;
   private String myMainComponentBinding;
-  private final ArrayList myButtonGroups = new ArrayList();
-  private final ArrayList myInspectionSuppressions = new ArrayList();
+  private final ArrayList<LwButtonGroup> myButtonGroups = new ArrayList<LwButtonGroup>();
+  private final ArrayList<LwInspectionSuppression> myInspectionSuppressions = new ArrayList<LwInspectionSuppression>();
 
-  public LwRootContainer() throws Exception{
+  public LwRootContainer() {
     super("javax.swing.JPanel");
     myLayoutSerializer = XYLayoutSerializer.INSTANCE;
   }
@@ -70,8 +69,7 @@ public final class LwRootContainer extends LwContainer implements IRootContainer
     myClassToBind = element.getAttributeValue(UIFormXmlConstants.ATTRIBUTE_BIND_TO_CLASS);
 
     // Constraints and properties
-    for(Iterator i=element.getChildren().iterator(); i.hasNext();){
-      final Element child = (Element)i.next();
+    for (final Element child : element.getChildren()) {
       if (child.getName().equals(UIFormXmlConstants.ELEMENT_BUTTON_GROUPS)) {
         readButtonGroups(child);
       }
@@ -89,8 +87,7 @@ public final class LwRootContainer extends LwContainer implements IRootContainer
   }
 
   private void readButtonGroups(final Element element) {
-    for(Iterator i=element.getChildren().iterator(); i.hasNext();){
-      final Element child = (Element)i.next();
+    for (final Element child : element.getChildren()) {
       LwButtonGroup group = new LwButtonGroup();
       group.read(child);
       myButtonGroups.add(group);
@@ -98,8 +95,7 @@ public final class LwRootContainer extends LwContainer implements IRootContainer
   }
 
   private void readInspectionSuppressions(final Element element) {
-    for(Iterator i=element.getChildren().iterator(); i.hasNext();){
-      final Element child = (Element)i.next();
+    for (final Element child : element.getChildren()) {
       String inspectionId = LwXmlReader.getRequiredString(child, UIFormXmlConstants.ATTRIBUTE_INSPECTION);
       String componentId = LwXmlReader.getString(child, UIFormXmlConstants.ATTRIBUTE_ID);
       myInspectionSuppressions.add(new LwInspectionSuppression(inspectionId, componentId));
@@ -108,16 +104,15 @@ public final class LwRootContainer extends LwContainer implements IRootContainer
 
   @Override
   public IButtonGroup[] getButtonGroups() {
-    return (LwButtonGroup[])myButtonGroups.toArray(new LwButtonGroup[0]);
+    return myButtonGroups.toArray(new LwButtonGroup[0]);
   }
 
   @Override
   public String getButtonGroupName(IComponent component) {
-    for(int i=0; i<myButtonGroups.size(); i++) {
-      LwButtonGroup group = (LwButtonGroup) myButtonGroups.get(i);
+    for (LwButtonGroup group : myButtonGroups) {
       final String[] ids = group.getComponentIds();
-      for(int j=0; j<ids.length; j++) {
-        if (ids [j].equals(component.getId())) {
+      for (String id : ids) {
+        if (id.equals(component.getId())) {
           return group.getName();
         }
       }
@@ -127,8 +122,7 @@ public final class LwRootContainer extends LwContainer implements IRootContainer
 
   @Override
   public String[] getButtonGroupComponentIds(String groupName) {
-    for(int i=0; i<myButtonGroups.size(); i++) {
-      LwButtonGroup group = (LwButtonGroup) myButtonGroups.get(i);
+    for (LwButtonGroup group : myButtonGroups) {
       if (group.getName().equals(groupName)) {
         return group.getComponentIds();
       }
@@ -138,8 +132,7 @@ public final class LwRootContainer extends LwContainer implements IRootContainer
 
   @Override
   public boolean isInspectionSuppressed(final String inspectionId, final String componentId) {
-    for (Iterator iterator = myInspectionSuppressions.iterator(); iterator.hasNext();) {
-      LwInspectionSuppression suppression = (LwInspectionSuppression)iterator.next();
+    for (LwInspectionSuppression suppression : myInspectionSuppressions) {
       if ((suppression.getComponentId() == null || suppression.getComponentId().equals(componentId)) &&
           suppression.getInspectionId().equals(inspectionId)) {
         return true;
@@ -149,6 +142,6 @@ public final class LwRootContainer extends LwContainer implements IRootContainer
   }
 
   public LwInspectionSuppression[] getInspectionSuppressions() {
-    return (LwInspectionSuppression[]) myInspectionSuppressions.toArray(LwInspectionSuppression.EMPTY_ARRAY);
+    return myInspectionSuppressions.toArray(LwInspectionSuppression.EMPTY_ARRAY);
   }
 }
index fe72072ab8e824e94fe08dd3997debfa621f8e03..1faf1e4991a00bdd3682a593bdfb736a58ff369c 100644 (file)
@@ -92,8 +92,8 @@ public final class BorderType {
 
   public static BorderType valueOf(final String name){
     BorderType[] allTypes = getAllTypes();
-    for(int i=0; i<allTypes.length; i++) {
-      if (allTypes [i].getId().equals(name)) return allTypes [i];
+    for (BorderType type : allTypes) {
+      if (type.getId().equals(name)) return type;
     }
     throw new UnexpectedFormElementException("unknown type: "+name);
   }
index 1785247b0763992e31549ed0fa165fe70dbef5a5..d2890a55faf606b41d869861f681325bdfddc48c 100644 (file)
@@ -890,8 +890,7 @@ public class GridBagConverterTest extends TestCase {
 
   private static void applyConversionResults(final JPanel panel, final GridBagConverter converter) {
     GridBagConverter.Result[] results = converter.convert();
-    for(int i=0; i<results.length; i++)  {
-      GridBagConverter.Result result = results [i];
+    for (GridBagConverter.Result result : results) {
       JComponent component = result.isFillerPanel ? new JPanel() : result.component;
       if (result.minimumSize != null) {
         component.setMinimumSize(result.minimumSize);
index 9a9c6c6ecb8813b39b0e014b8890fe7ad8d7c9ce..2af14b602b13d4d6f5f955ccf6d89a0a82dbf51d 100644 (file)
@@ -36,8 +36,8 @@ import java.util.*;
 
 public class Javac2 extends Javac {
   public static final String PROPERTY_INSTRUMENTATION_INCLUDE_JAVA_RUNTIME = "javac2.instrumentation.includeJavaRuntime";
-  private ArrayList myFormFiles;
-  private List myNestedFormPathList;
+  private ArrayList<File> myFormFiles;
+  private List<PrefixedPath> myNestedFormPathList;
   private boolean instrumentNotNull = true;
   private String myNotNullAnnotations = "org.jetbrains.annotations.NotNull";
   private final List<Regexp> myClassFilterAnnotationRegexpList = new ArrayList<Regexp>(0);
@@ -235,7 +235,7 @@ public class Javac2 extends Javac {
   public PrefixedPath createNestedformdirs() {
     PrefixedPath p = new PrefixedPath(getProject());
     if (myNestedFormPathList == null) {
-      myNestedFormPathList = new ArrayList();
+      myNestedFormPathList = new ArrayList<PrefixedPath>();
     }
     myNestedFormPathList.add(p);
     return p;
@@ -279,17 +279,16 @@ public class Javac2 extends Javac {
    */
   private void instrumentForms(final InstrumentationClassFinder finder) {
     // we instrument every file, because we cannot find which files should not be instrumented without dependency storage
-    final ArrayList formsToInstrument = myFormFiles;
+    final ArrayList<File> formsToInstrument = myFormFiles;
 
-    if (formsToInstrument.size() == 0) {
+    if (formsToInstrument.isEmpty()) {
       log("No forms to instrument found", Project.MSG_VERBOSE);
       return;
     }
 
-    final HashMap class2form = new HashMap();
+    final HashMap<String, File> class2form = new HashMap<String, File>();
 
-    for (int i = 0; i < formsToInstrument.size(); i++) {
-      final File formFile = (File)formsToInstrument.get(i);
+    for (File formFile : formsToInstrument) {
 
       log("compiling form " + formFile.getAbsolutePath(), Project.MSG_VERBOSE);
       final LwRootContainer rootContainer;
@@ -317,7 +316,7 @@ public class Javac2 extends Javac {
         continue;
       }
 
-      final File alreadyProcessedForm = (File)class2form.get(classToBind);
+      final File alreadyProcessedForm = class2form.get(classToBind);
       if (alreadyProcessedForm != null) {
         fireError(formFile.getAbsolutePath() +
                   ": " +
@@ -346,17 +345,17 @@ public class Javac2 extends Javac {
         codeGenerator.patchFile(classFile);
         final FormErrorInfo[] warnings = codeGenerator.getWarnings();
 
-        for (int j = 0; j < warnings.length; j++) {
-          log(formFile.getAbsolutePath() + ": " + warnings[j].getErrorMessage(), Project.MSG_WARN);
+        for (FormErrorInfo warning : warnings) {
+          log(formFile.getAbsolutePath() + ": " + warning.getErrorMessage(), Project.MSG_WARN);
         }
         final FormErrorInfo[] errors = codeGenerator.getErrors();
         if (errors.length > 0) {
           StringBuilder message = new StringBuilder();
-          for (int j = 0; j < errors.length; j++) {
+          for (FormErrorInfo error : errors) {
             if (message.length() > 0) {
               message.append("\n");
             }
-            message.append(formFile.getAbsolutePath()).append(": ").append(errors[j].getErrorMessage());
+            message.append(formFile.getAbsolutePath()).append(": ").append(error.getErrorMessage());
           }
           fireError(message.toString());
         }
@@ -408,8 +407,7 @@ public class Javac2 extends Javac {
     cp.addExtdirs(getExtdirs());
 
     final String[] pathElements = cp.list();
-    for (int i = 0; i < pathElements.length; i++) {
-      final String pathElement = pathElements[i];
+    for (final String pathElement : pathElements) {
       classPathBuffer.append(File.pathSeparator);
       classPathBuffer.append(pathElement);
     }
@@ -462,8 +460,7 @@ public class Javac2 extends Javac {
   private int instrumentNotNull(File dir, final InstrumentationClassFinder finder) {
     int instrumented = 0;
     final File[] files = dir.listFiles();
-    for (int i = 0; i < files.length; i++) {
-      File file = files[i];
+    for (File file : files) {
       final String name = file.getName();
       if (name.endsWith(".class")) {
         final String path = file.getPath();
@@ -526,7 +523,7 @@ public class Javac2 extends Javac {
       return false;
     }
 
-    final boolean[] result = new boolean[]{false};
+    final boolean[] result = {false};
     reader.accept(new ClassVisitor(Opcodes.API_VERSION) {
       @Override
       public AnnotationVisitor visitAnnotation(String desc, boolean visible) {
@@ -572,14 +569,13 @@ public class Javac2 extends Javac {
   @Override
   protected void resetFileLists() {
     super.resetFileLists();
-    myFormFiles = new ArrayList();
+    myFormFiles = new ArrayList<File>();
   }
 
   @Override
   protected void scanDir(final File srcDir, final File destDir, final String[] files) {
     super.scanDir(srcDir, destDir, files);
-    for (int i = 0; i < files.length; i++) {
-      final String file = files[i];
+    for (final String file : files) {
       if (file.endsWith(".form")) {
         log("Found form file " + file, Project.MSG_VERBOSE);
         myFormFiles.add(new File(srcDir, file));
@@ -588,7 +584,7 @@ public class Javac2 extends Javac {
   }
 
   private static InstrumentationClassFinder createInstrumentationClassFinder(final String classPath, boolean shouldIncludeJavaRuntime) throws MalformedURLException {
-    final ArrayList urls = new ArrayList();
+    final ArrayList<URL> urls = new ArrayList<URL>();
     if (shouldIncludeJavaRuntime) {
       final URL jrt = tryGetJrtURL();
       if (jrt != null) {
@@ -599,14 +595,14 @@ public class Javac2 extends Javac {
       final String s = tokenizer.nextToken();
       urls.add(new File(s).toURI().toURL());
     }
-    final URL[] urlsArr = (URL[])urls.toArray(new URL[0]);
+    final URL[] urlsArr = urls.toArray(new URL[0]);
     return new InstrumentationClassFinder(urlsArr);
   }
 
   private class AntNestedFormLoader implements NestedFormLoader {
     private final ClassLoader myLoader;
-    private final List myNestedFormPathList;
-    private final HashMap myFormCache = new HashMap();
+    private final List<PrefixedPath> myNestedFormPathList;
+    private final HashMap<String, LwRootContainer> myFormCache = new HashMap<String, LwRootContainer>();
 
     public AntNestedFormLoader(final ClassLoader loader, List nestedFormPathList) {
       myLoader = loader;
@@ -616,13 +612,12 @@ public class Javac2 extends Javac {
     @Override
     public LwRootContainer loadForm(String formFilePath) throws Exception {
       if (myFormCache.containsKey(formFilePath)) {
-        return (LwRootContainer)myFormCache.get(formFilePath);
+        return myFormCache.get(formFilePath);
       }
 
       String lowerFormFilePath = formFilePath.toLowerCase();
       log("Searching for form " + lowerFormFilePath, Project.MSG_VERBOSE);
-      for (Iterator iterator = myFormFiles.iterator(); iterator.hasNext();) {
-        File file = (File)iterator.next();
+      for (File file : myFormFiles) {
         String name = file.getAbsolutePath().replace(File.separatorChar, '/').toLowerCase();
         log("Comparing with " + name, Project.MSG_VERBOSE);
         if (name.endsWith(lowerFormFilePath)) {
@@ -631,8 +626,7 @@ public class Javac2 extends Javac {
       }
 
       if (myNestedFormPathList != null) {
-        for (int i = 0; i < myNestedFormPathList.size(); i++) {
-          PrefixedPath path = (PrefixedPath)myNestedFormPathList.get(i);
+        for (PrefixedPath path : myNestedFormPathList) {
           File formFile = path.findFile(formFilePath);
           if (formFile != null) {
             return loadForm(formFilePath, new FileInputStream(formFile));
index c7af4eb63613c11e0240dc1bb50b2f3d353e3620..25a58eb1bd30161bcd37a2fdcb7c2c7f214463ae 100644 (file)
@@ -57,8 +57,8 @@ public class PrefixedPath extends Path {
     }
 
     String[] dirsList = list();
-    for (int j = 0, listLength = dirsList.length; j < listLength; j++) {
-      String fullPath = ensureEndsWithSlash(dirsList[j]) + relativePath;
+    for (String aDirsList : dirsList) {
+      String fullPath = ensureEndsWithSlash(aDirsList) + relativePath;
       File file = new File(fullPath.replace('/', File.separatorChar));
       if (file.isFile()) {
         return file;
index 8d88cd5160601d9ce387538d74a3854a4370fa80..6f2d94eef1fac91d92c61c5902cbfe2cee8115c3 100644 (file)
@@ -76,8 +76,7 @@ public class MethodListDlg extends DialogWrapper {
   }
 
   private void createList(final PsiMethod[] allMethods, final Condition<? super PsiMethod> filter) {
-    for (int i = 0; i < allMethods.length; i++) {
-      final PsiMethod method = allMethods[i];
+    for (final PsiMethod method : allMethods) {
       if (filter.value(method)) myListModel.add(method);
     }
   }
index cef68519481d72338c457d53d589c5b1c30e8c8d..f57d3c8f63a4fcfc2e923776c0ddb8cf98fb599e 100644 (file)
@@ -86,9 +86,9 @@ public class LiteAnalyzer<V extends Value> implements Opcodes {
       Type ctype = Type.getObjectType(owner);
       current.setLocal(local++, interpreter.newValue(ctype));
     }
-    for (int i = 0; i < args.length; ++i) {
-      current.setLocal(local++, interpreter.newValue(args[i]));
-      if (args[i].getSize() == 2) {
+    for (Type arg : args) {
+      current.setLocal(local++, interpreter.newValue(arg));
+      if (arg.getSize() == 2) {
         current.setLocal(local++, interpreter.newValue(null));
       }
     }
index 95ac3f57e252729aabdd7dabf9a39583baeca88b..b7977cca3a98755f16256daf7a85a0d399f7bee6 100644 (file)
@@ -92,9 +92,9 @@ public class LiteAnalyzerExt<V extends Value, Data, MyInterpreter extends Interp
       Type ctype = Type.getObjectType(owner);
       current.setLocal(local++, interpreter.newValue(ctype));
     }
-    for (int i = 0; i < args.length; ++i) {
-      current.setLocal(local++, interpreter.newValue(args[i]));
-      if (args[i].getSize() == 2) {
+    for (Type arg : args) {
+      current.setLocal(local++, interpreter.newValue(arg));
+      if (arg.getSize() == 2) {
         current.setLocal(local++, interpreter.newValue(null));
       }
     }
index f852ca5a5a497ae98503ca8934351384942cfb9a..c7086efc2f382c2d16b9656673b638fd1fa4a2cf 100644 (file)
@@ -259,8 +259,7 @@ public class DeadHTMLComposer extends HTMLComposerImpl {
   private static int getDerivedRefsCount(RefMethod refMethod) {
     int count = 0;
 
-    for (Iterator<RefMethod> iterator = refMethod.getDerivedMethods().iterator(); iterator.hasNext();) {
-      RefMethod refDerived = iterator.next();
+    for (RefMethod refDerived : refMethod.getDerivedMethods()) {
       count += refDerived.getInReferences().size() + getDerivedRefsCount(refDerived);
     }
 
@@ -270,8 +269,7 @@ public class DeadHTMLComposer extends HTMLComposerImpl {
   private static int getSuperRefsCount(RefMethod refMethod) {
     int count = 0;
 
-    for (Iterator<RefMethod> iterator = refMethod.getSuperMethods().iterator(); iterator.hasNext();) {
-      RefMethod refSuper = iterator.next();
+    for (RefMethod refSuper : refMethod.getSuperMethods()) {
       count += refSuper.getInReferences().size() + getSuperRefsCount(refSuper);
     }
 
@@ -282,13 +280,11 @@ public class DeadHTMLComposer extends HTMLComposerImpl {
     if (!aClass.isAnonymous()) {
       int count = 0;
 
-      for (Iterator<RefMethod> iterator = aClass.getConstructors().iterator(); iterator.hasNext();) {
-        RefMethod refConstructor = iterator.next();
+      for (RefMethod refConstructor : aClass.getConstructors()) {
         count += refConstructor.getInReferences().size();
       }
 
-      for (Iterator<RefClass> iterator = aClass.getSubClasses().iterator(); iterator.hasNext();) {
-        RefClass subClass = iterator.next();
+      for (RefClass subClass : aClass.getSubClasses()) {
         count += getInstantiationsCount(subClass);
         count -= subClass.getConstructors().size();
       }
@@ -301,8 +297,7 @@ public class DeadHTMLComposer extends HTMLComposerImpl {
 
   private static int getImplementationsCount(RefClass refClass) {
     int count = 0;
-    for (Iterator<RefClass> iterator = refClass.getSubClasses().iterator(); iterator.hasNext();) {
-      RefClass subClass = iterator.next();
+    for (RefClass subClass : refClass.getSubClasses()) {
       if (!subClass.isInterface() && !subClass.isAbstract()) {
         count++;
       }
@@ -319,10 +314,8 @@ public class DeadHTMLComposer extends HTMLComposerImpl {
       appendHeading(buf, InspectionsBundle.message("inspection.dead.code.export.results.instantiated.from.heading"));
 
       startList(buf);
-      for (Iterator<RefMethod> iterator = refClass.getConstructors().iterator(); iterator.hasNext();) {
-        RefMethod refMethod = iterator.next();
-        for (Iterator<RefElement> constructorCallersIterator = refMethod.getInReferences().iterator(); constructorCallersIterator.hasNext();) {
-          RefElement refCaller = constructorCallersIterator.next();
+      for (RefMethod refMethod : refClass.getConstructors()) {
+        for (RefElement refCaller : refMethod.getInReferences()) {
           appendListItem(buf, refCaller);
           found = true;
         }
index 876e2250da1f3235cbae8f610026e2b1aec0a970..dec004093f0567cd697687b16691adab0e4963ab 100644 (file)
@@ -207,14 +207,13 @@ public class CyclicDependenciesBuilder{
       myGraph = buildGraph();
     }
     final HashMap<PsiPackage, Set<List<PsiPackage>>> result = new HashMap<>();
-    for (Iterator<PsiPackage> iterator = packages.iterator(); iterator.hasNext();) {
-      PsiPackage psiPackage = iterator.next();
-        Set<List<PsiPackage>> paths2Pack = result.get(psiPackage);
-        if (paths2Pack == null) {
-          paths2Pack = new HashSet<>();
-          result.put(psiPackage, paths2Pack);
-        }
-        paths2Pack.addAll(GraphAlgorithms.getInstance().findCycles(myGraph, psiPackage));
+    for (PsiPackage psiPackage : packages) {
+      Set<List<PsiPackage>> paths2Pack = result.get(psiPackage);
+      if (paths2Pack == null) {
+        paths2Pack = new HashSet<>();
+        result.put(psiPackage, paths2Pack);
+      }
+      paths2Pack.addAll(GraphAlgorithms.getInstance().findCycles(myGraph, psiPackage));
     }
     return result;
   }
index 54449b295f6b752a3555bafe07d6f196033f4581..e33f278612823b7a611394c7f6b253debfd15fcc 100644 (file)
@@ -56,8 +56,7 @@ public final class TypeHierarchyTreeStructure extends SubtypesHierarchyTreeStruc
       final PsiClass aClass1 = aClass;
       final PsiClass[] superTypes = aClass1.getSupers();
       PsiClass superType = null;
-      for (int i = 0; i < superTypes.length; i++) {
-        final PsiClass type = superTypes[i];
+      for (final PsiClass type : superTypes) {
         if (!type.isInterface()) {
           superType = type;
           break;
index a20d7af15fe83dc25f0508d4766b6bd201bc4638..48cf243caff09e6c6a2fa61301597bed6fbd15f0 100644 (file)
@@ -70,11 +70,11 @@ public class GlobalAnalyzer {
 
     final LinkedList<T> worklist = new LinkedList<>();
 
-    for (Iterator<T> i = init.iterator(); i.hasNext();) {
-      result = stepOneEnd(i.next(), worklist, functor) || result;
+    for (T anInit : init) {
+      result = stepOneEnd(anInit, worklist, functor) || result;
     }
 
-    while (worklist.size() > 0) {
+    while (!worklist.isEmpty()) {
       result = stepOneEnd(worklist.removeFirst(), worklist, functor) || result;
     }
 
@@ -86,11 +86,11 @@ public class GlobalAnalyzer {
 
     final LinkedList<T> worklist = new LinkedList<>();
 
-    for (Iterator<T> i = init.iterator(); i.hasNext();) {
-      result = stepTwoEnds(i.next(), worklist, functor) || result;
+    for (T anInit : init) {
+      result = stepTwoEnds(anInit, worklist, functor) || result;
     }
 
-    while (worklist.size() > 0) {
+    while (!worklist.isEmpty()) {
       result = stepTwoEnds(worklist.removeFirst(), worklist, functor) || result;
     }
 
index a431cc602611137097b8e41d694c76af539c9b1a..6fe6a6c6eb2c4301e329dfadfcd1e936dc7fc944 100644 (file)
@@ -52,8 +52,8 @@ public class EncapsulateFieldsAction extends BaseRefactoringAction {
       return elements[0] instanceof PsiClass && elements[0].getLanguage().isKindOf(JavaLanguage.INSTANCE) || isAcceptedField(elements[0]);
     }
     else if (elements.length > 1) {
-      for (int idx = 0; idx < elements.length; idx++) {
-        if (!isAcceptedField(elements[idx])) {
+      for (PsiElement element : elements) {
+        if (!isAcceptedField(element)) {
           return false;
         }
       }
index 3629724c82902e1149747e8603fd9d5ad6fd256f..4801b7ecf80603dd1912a31231f707efe7fe19f8 100644 (file)
@@ -41,8 +41,8 @@ public class InnerClassConstructor extends InnerClassMethod {
 
     PsiExpressionList arguments = ((PsiMethodCallExpression) superCallStatement.getExpression()).getArgumentList();
     PsiParameter[] parameters = parameterList.getParameters();
-    for (int i = 0; i < parameters.length; i++) {
-      arguments.add(factory.createExpressionFromText(parameters[i].getName(), null));
+    for (PsiParameter parameter : parameters) {
+      arguments.add(factory.createExpressionFromText(parameter.getName(), null));
     }
     constructor.getBody().add(superCallStatement);
     innerClass.add(constructor);
index a230f99ed746e5938223e6bc0aaab1bc102d6f12..91e4a81c384f0492650a975dd8dfae7a78da781c 100644 (file)
@@ -31,8 +31,7 @@ public class MigrationMap {
   }
 
   public MigrationMap(MigrationMapEntry[] entries) {
-    for (int i = 0; i < entries.length; i++) {
-      MigrationMapEntry entry = entries[i];
+    for (MigrationMapEntry entry : entries) {
       addEntry(entry);
     }
   }
index 62092725c0a113b4be67cdd3aff7a959f8cecca7..b24a1566e4b163056ed75bb73addb63e0781afb7 100644 (file)
@@ -200,16 +200,16 @@ public class MigrationMapSet {
     copyPredefinedMaps(dir);
 
     File[] files = getMapFiles(dir);
-    for(int i = 0; i < files.length; i++){
-      try{
-        MigrationMap map = readMap(files[i]);
-        if (map != null){
-          map.setFileName(FileUtil.getNameWithoutExtension(files[i]));
+    for (File file : files) {
+      try {
+        MigrationMap map = readMap(file);
+        if (map != null) {
+          map.setFileName(FileUtil.getNameWithoutExtension(file));
           myMaps.add(map);
         }
       }
-      catch(InvalidDataException | JDOMException e){
-        LOG.error("Invalid data in file: " + files[i].getAbsolutePath());
+      catch (InvalidDataException | JDOMException e) {
+        LOG.error("Invalid data in file: " + file.getAbsolutePath());
       }
       catch (IOException e) {
         LOG.error(e);
@@ -229,41 +229,40 @@ public class MigrationMapSet {
 
     MigrationMap map = new MigrationMap();
 
-    for(Iterator i = root.getChildren().iterator(); i.hasNext(); ){
-      Element node = (Element)i.next();
-      if (NAME.equals(node.getName())){
+    for (Element node : root.getChildren()) {
+      if (NAME.equals(node.getName())) {
         String name = node.getAttributeValue(VALUE);
         map.setName(name);
       }
-      if (DESCRIPTION.equals(node.getName())){
+      if (DESCRIPTION.equals(node.getName())) {
         String description = node.getAttributeValue(VALUE);
         map.setDescription(description);
       }
 
-      if (ENTRY.equals(node.getName())){
+      if (ENTRY.equals(node.getName())) {
         MigrationMapEntry entry = new MigrationMapEntry();
         String oldName = node.getAttributeValue(OLD_NAME);
-        if (oldName == null){
+        if (oldName == null) {
           throw new InvalidDataException();
         }
         entry.setOldName(oldName);
         String newName = node.getAttributeValue(NEW_NAME);
-        if (newName == null){
+        if (newName == null) {
           throw new InvalidDataException();
         }
         entry.setNewName(newName);
         String typeStr = node.getAttributeValue(TYPE);
-        if (typeStr == null){
+        if (typeStr == null) {
           throw new InvalidDataException();
         }
         entry.setType(MigrationMapEntry.CLASS);
-        if (typeStr.equals(PACKAGE_TYPE)){
+        if (typeStr.equals(PACKAGE_TYPE)) {
           entry.setType(MigrationMapEntry.PACKAGE);
           @NonNls String isRecursiveStr = node.getAttributeValue(RECURSIVE);
-          if (isRecursiveStr != null && isRecursiveStr.equals("true")){
+          if (isRecursiveStr != null && isRecursiveStr.equals("true")) {
             entry.setRecursive(true);
           }
-          else{
+          else {
             entry.setRecursive(false);
           }
         }
index faa9cdc1910dc2698e7d165110094a8093c3d5c1..6769a737848583ca4f825ddb372081dff009430b 100644 (file)
@@ -35,8 +35,8 @@ public abstract class ClassReferenceScanner {
       myReferences = findReferences();
     }
 
-    for (int i = 0; i < myReferences.length; i++) {
-      processUsage(myReferences[i].getElement(), visitor);
+    for (PsiReference reference : myReferences) {
+      processUsage(reference.getElement(), visitor);
     }
   }
 
index fd2f96e648c31f3ef56431eace52c42c9b142a9a..f533dcf0a9fc75c4f305e83750276d6ccf52ffba 100644 (file)
@@ -59,8 +59,7 @@ public abstract class FixMethodJavadocTest extends FileSetTestCase {
   private void collectNewParameters(PsiMethod method, String[] names, Set<PsiParameter> newParameters) {
     Set<String> newNames = new HashSet<>(Arrays.asList(names));
     final PsiParameter[] parameters = method.getParameterList().getParameters();
-    for (int i = 0; i < parameters.length; i++) {
-      PsiParameter parameter = parameters[i];
+    for (PsiParameter parameter : parameters) {
       if (newNames.contains(parameter.getName())) {
         newParameters.add(parameter);
       }
index c67ec28bce9d51efc68c7ac7c9de1a6849306458..eba62db3355485bcf801a024d583bcee82f03b7f 100644 (file)
@@ -29,7 +29,6 @@ import org.jetbrains.annotations.NotNull;
 
 import java.util.ArrayList;
 import java.util.Collection;
-import java.util.Iterator;
 
 class SameNamesJoiner implements TreeStructureProvider {
   @NotNull
@@ -40,8 +39,8 @@ class SameNamesJoiner implements TreeStructureProvider {
     ArrayList<AbstractTreeNode> result = new ArrayList<>();
 
     MultiValuesMap<Object, AbstractTreeNode> executed = new MultiValuesMap<>();
-    for (Iterator<AbstractTreeNode> iterator = children.iterator(); iterator.hasNext();) {
-      ProjectViewNode treeNode = (ProjectViewNode)iterator.next();
+    for (AbstractTreeNode child : children) {
+      ProjectViewNode treeNode = (ProjectViewNode)child;
       Object o = treeNode.getValue();
       if (o instanceof PsiFile) {
         String name = ((PsiFile)o).getVirtualFile().getNameWithoutExtension();
@@ -52,9 +51,7 @@ class SameNamesJoiner implements TreeStructureProvider {
       }
     }
 
-    Iterator<Object> keys = executed.keySet().iterator();
-    while (keys.hasNext()) {
-      Object each = keys.next();
+    for (Object each : executed.keySet()) {
       Collection<AbstractTreeNode> objects = executed.get(each);
       if (objects.size() > 1) {
         result.add(new JoinedNode(objects, new Joined(findPsiFileIn(objects))));
@@ -72,8 +69,7 @@ class SameNamesJoiner implements TreeStructureProvider {
   }
 
   private PsiFile findPsiFileIn(Collection<AbstractTreeNode> objects) {
-    for (Iterator<AbstractTreeNode> iterator = objects.iterator(); iterator.hasNext();) {
-      AbstractTreeNode treeNode = iterator.next();
+    for (AbstractTreeNode treeNode : objects) {
       if (treeNode.getValue() instanceof PsiFile) return (PsiFile)treeNode.getValue();
     }
     return null;
@@ -82,10 +78,9 @@ class SameNamesJoiner implements TreeStructureProvider {
   private boolean hasElementWithTheSameName(PsiFile element) {
     PsiDirectory psiDirectory = element.getParent();
     PsiElement[] children = psiDirectory.getChildren();
-    for (int i = 0; i < children.length; i++) {
-      PsiElement child = children[i];
-
-      if (child != element && element.getVirtualFile().getNameWithoutExtension().equals(((PsiFile)child).getVirtualFile().getNameWithoutExtension())){
+    for (PsiElement child : children) {
+      if (child != element &&
+          element.getVirtualFile().getNameWithoutExtension().equals(((PsiFile)child).getVirtualFile().getNameWithoutExtension())) {
         return true;
       }
     }
index 511f63bdaded22e8adc4ee571e4b8d866d77ac19..6bca73f205e64b30cc59474e69428e2cc7aefdae 100644 (file)
@@ -42,9 +42,9 @@ public class PatchedJar extends Jar {
         // checks here defering them for the second run
         Resource[][] manifests = grabManifests(rcs);
         int count = 0;
-        for (int i = 0; i < manifests.length; i++) {
-            count += manifests[i].length;
-        }
+      for (Resource[] manifest : manifests) {
+        count += manifest.length;
+      }
         log("found a total of " + count + " manifests in "
             + manifests.length + " resource collections",
             Project.MSG_VERBOSE);
index 2b11b3ed4395c9924297dc0efdc3be30852d9ceb..221c4551accc2fb573a3a369fffed1d2cb6a83b3 100644 (file)
@@ -90,8 +90,8 @@ public class ProjectListBuilder extends AbstractListBuilder {
   protected List<AbstractTreeNode> getAllAcceptableNodes(final Object[] childElements, VirtualFile file) {
     ArrayList<AbstractTreeNode> result = new ArrayList<>();
 
-    for (int i = 0; i < childElements.length; i++) {
-      ProjectViewNode childElement = (ProjectViewNode)childElements[i];
+    for (Object childElement1 : childElements) {
+      ProjectViewNode childElement = (ProjectViewNode)childElement1;
       if (childElement.contains(file)) result.add(childElement);
     }
 
index c72f5232361eba4056def82a21cb2245edbda099..bd53019af6d20f469f675d4f883c048e91efaee2 100644 (file)
@@ -91,8 +91,7 @@ public class FilterInfo implements JDOMExternalizable {
 
   @Override
   public void readExternal(Element element) {
-    for (Iterator i2 = element.getChildren(ELEMENT_OPTION).iterator(); i2.hasNext(); ) {
-      Element optionElement = (Element)i2.next();
+    for (Element optionElement : element.getChildren(ELEMENT_OPTION)) {
       String value = optionElement.getAttributeValue(ATTRIBUTE_VALUE);
       String name = optionElement.getAttributeValue(ATTRIBUTE_NAME);
 
index 21b5f53d3718b2d618434356e0609311946df511..7db822a691380bcd2876106d7f12fe15ec4984fc 100644 (file)
@@ -341,8 +341,8 @@ public class MultiLineLabelUI extends BasicLabelUI {
 
   public static Dimension computeMultiLineDimension(FontMetrics fm, String[] strs) {
     int width = 0;
-    for (int i = 0; i < strs.length; i++) {
-      width = Math.max(width, SwingUtilities.computeStringWidth(fm, strs[i]));
+    for (String str : strs) {
+      width = Math.max(width, SwingUtilities.computeStringWidth(fm, str));
     }
     return new Dimension(width, fm.getHeight() * strs.length);
   }
index 10b8c3c64050de4c11a2c6411e0029780e8f34ab..90a51d792539d67859528c77ec38b60fe5a1510c 100644 (file)
@@ -96,8 +96,8 @@ public class ListUtil {
     int firstSelectedIndex = idxs[0];
     ArrayList<T> removedItems = new ArrayList<>();
     int deletedCount = 0;
-    for (int idx = 0; idx < idxs.length; idx++) {
-      int index = idxs[idx] - deletedCount;
+    for (int idx1 : idxs) {
+      int index = idx1 - deletedCount;
       if (index < 0 || index >= model.getSize()) continue;
       T obj = (T)get(model, index);
       if (condition == null || condition.value(obj)) {
@@ -132,8 +132,7 @@ public class ListUtil {
       return false;
     }
 
-    for (int idx = 0; idx < idxs.length; idx++) {
-      int index = idxs[idx];
+    for (int index : idxs) {
       if (index < 0 || index >= model.getSize()) continue;
       Object obj = getExtensions(model).get(model, index);
       if (applyable == null || applyable.value(obj)) {
@@ -148,8 +147,7 @@ public class ListUtil {
     DefaultListModel model = getModel(list);
     int[] indices = list.getSelectedIndices();
     if (!canMoveSelectedItemsUp(list)) return 0;
-    for(int i = 0; i < indices.length; i++){
-      int index = indices[i];
+    for (int index : indices) {
       Object temp = model.get(index);
       model.set(index, model.get(index - 1));
       model.set(index - 1, temp);
index 7005bfff78cf02ef7f84bfa25dbf8511a996b141..5e11daa53cb127d57241ed5ca209b67b838626dc 100644 (file)
@@ -140,8 +140,7 @@ public abstract class ReorderableListController <T> {
     }
 
     protected void runPostHandlers(final V change) {
-      for (Iterator<ActionNotification<V>> iterator = myPostHandlers.iterator(); iterator.hasNext();) {
-        final ActionNotification<V> runnable = iterator.next();
+      for (final ActionNotification<V> runnable : myPostHandlers) {
         runnable.afterActionPerformed(change);
       }
     }
index 2f2f4800037c670638ef0e9144af0dda880b35b6..ab4b324221a42ba6a8583f2e46f4fa64ab626dfd 100644 (file)
@@ -92,15 +92,13 @@ public abstract class AbstractProperty<T> {
     }
 
     public final void copyFrom(AbstractPropertyContainer source, AbstractProperty[] properties) {
-      for (int i = 0; i < properties.length; i++) {
-        AbstractProperty property = properties[i];
+      for (AbstractProperty property : properties) {
         setValueOf((PropertyImpl)property, source.getValueOf(property));
       }
     }
 
     public final boolean areValueEqual(AbstractPropertyContainer other, AbstractProperty[] properties) {
-      for (int i = 0; i < properties.length; i++) {
-        AbstractProperty property = properties[i];
+      for (AbstractProperty property : properties) {
         if (!property.areEqual(getValueOf((PropertyImpl)property), other.getValueOf(property))) return false;
       }
       return true;
index 62dc641c929fd673d264f88d0b4095c6fe18a17b..5e56dc332f844c927ba90230787a8792ed213f70 100644 (file)
@@ -20,8 +20,7 @@ public class PreviousDiffAction extends DiffWalkerAction {
     int[] fragmentBeginnings = side.getFragmentStartingLines();
     int gotoLine = -1;
     if (fragmentBeginnings == null) return -1;
-    for (int i = 0; i < fragmentBeginnings.length; i++) {
-      int line = fragmentBeginnings[i];
+    for (int line : fragmentBeginnings) {
       if (line < editor.getCaretModel().getLogicalPosition().line) {
         gotoLine = line;
       }
index c695dcaf7427362dacd075f260995001301bc6ad..db105ac15d5884a86a366a7a8481231772594a4e 100644 (file)
@@ -113,10 +113,9 @@ public class MultiLevelDiffTool implements DiffTool, DiscloseMultiRequest {
   public static boolean canShowRequest(DiffRequest request) {
     boolean isFile = false;
     DiffContent[] contents = request.getContents();
-    for (int i = 0; i < contents.length; i++) {
-      DiffContent content = contents[i];
+    for (DiffContent content : contents) {
       VirtualFile file = content.getFile();
-      if (file != null && file.isInLocalFileSystem() && ! file.isDirectory()) {
+      if (file != null && file.isInLocalFileSystem() && !file.isDirectory()) {
         isFile = true;
         break;
       }
index 5e174d275ab1269fae193acd0c091c1993634e13..177128cb2f062df72a3527ac74c42f506f5a6edb 100644 (file)
 package com.intellij.openapi.keymap.impl;
 
 import com.intellij.openapi.util.InvalidDataException;
-import java.util.HashMap;
 import org.jdom.Element;
 import org.jetbrains.annotations.NonNls;
 
 import java.util.ArrayList;
+import java.util.HashMap;
 import java.util.Iterator;
 
 /**
@@ -90,7 +90,7 @@ public class Converter01{
     if(name==null){
       throw new InvalidDataException("Attribute 'name' of <keymap> must be specified");
     }
-    HashMap id2elements=new HashMap();
+    HashMap<String, ArrayList<Element>> id2elements = new HashMap<>();
 
     for(Iterator i=keymapElement.getChildren().iterator();i.hasNext();){
       Element oldChild=(Element)i.next();
@@ -111,14 +111,14 @@ public class Converter01{
             newChild.setAttribute(SECOND_KEYSTROKE_ATTRIBUTE,suffix);
           }
           // Put new child into the map
-          ArrayList elements=(ArrayList)id2elements.get(id);
+          ArrayList<Element> elements = id2elements.get(id);
           if(elements==null){
-            elements=new ArrayList(2);
+            elements = new ArrayList<>(2);
             id2elements.put(id,elements);
           }
           elements.add(newChild);
         }else{
-          id2elements.put(id,new ArrayList(0));
+          id2elements.put(id, new ArrayList<>(0));
         }
         // Remove old child
         i.remove();
@@ -132,9 +132,9 @@ public class Converter01{
         // Remove old child
         i.remove();
         // Put new child into the map
-        ArrayList elements=(ArrayList)id2elements.get(id);
+        ArrayList<Element> elements = id2elements.get(id);
         if(elements==null){
-          elements=new ArrayList(2);
+          elements = new ArrayList<>(2);
           id2elements.put(id,elements);
         }
         elements.add(oldChild);
@@ -143,13 +143,11 @@ public class Converter01{
       }
     }
 
-    for(Iterator i=id2elements.keySet().iterator();i.hasNext();){
-      String id=(String)i.next();
-      Element actionElement=new Element(ACTION);
-      actionElement.setAttribute(ID_ATTRIBUTE,id);
-      ArrayList elements=(ArrayList)id2elements.get(id);
-      for(Iterator j=elements.iterator();j.hasNext();){
-        Element newChild=(Element)j.next();
+    for (String id : id2elements.keySet()) {
+      Element actionElement = new Element(ACTION);
+      actionElement.setAttribute(ID_ATTRIBUTE, id);
+      ArrayList<Element> elements = id2elements.get(id);
+      for (Element newChild : elements) {
         actionElement.addContent(newChild);
       }
       keymapElement.addContent(actionElement);
index f780536456f05d9ffb5eb3115ce876b230368614..38b4e3f1ba63c729ae8e88c8c874b0ac6cd99622 100644 (file)
@@ -38,8 +38,7 @@ public class PathMacro {
   public File resolveFile(String path, File defaultDir) {
     Set<String> macros = myMap.keySet();
     String actualtPath = path;
-    for (Iterator<String> iterator = macros.iterator(); iterator.hasNext(); ) {
-      String each = iterator.next();
+    for (String each : macros) {
       actualtPath = actualtPath.replaceAll(each, myMap.get(each).getAbsolutePath());
     }
 
index 0f7329b6c3b368c1663ecbfd89f4a4c99cb704c3..c329d87a2d7a95fa7d07eb8e9ea8c8b4c5c62ec2 100644 (file)
@@ -33,8 +33,8 @@ public class MigrateToNewDiffUtil {
     String[] titles = oldRequest.getContentTitles();
     List<DiffContent> newContents = new ArrayList<>(contents.length);
 
-    for (int i = 0; i < contents.length; i++) {
-      newContents.add(convertContent(oldRequest.getProject(), contents[i]));
+    for (com.intellij.openapi.diff.DiffContent content : contents) {
+      newContents.add(convertContent(oldRequest.getProject(), content));
     }
 
     return new SimpleDiffRequest(oldRequest.getWindowTitle(), newContents, Arrays.asList(titles));
index 8f725c92c79dcf65ddf9333a8dae54cee1aa06c6..742f21201289372cd5a5e55ad4393ce21967a7e9 100644 (file)
@@ -474,9 +474,7 @@ public class ColorThief {
       int[] histo = new int[HISTOSIZE];
       int index, rval, gval, bval;
 
-      int numPixels = pixels.length;
-      for (int i = 0; i < numPixels; i++) {
-        int[] pixel = pixels[i];
+      for (int[] pixel : pixels) {
         rval = pixel[0] >> RSHIFT;
         gval = pixel[1] >> RSHIFT;
         bval = pixel[2] >> RSHIFT;
@@ -494,28 +492,29 @@ public class ColorThief {
       int rval, gval, bval;
 
       // find min/max
-      int numPixels = pixels.length;
-      for (int i = 0; i < numPixels; i++) {
-        int[] pixel = pixels[i];
+      for (int[] pixel : pixels) {
         rval = pixel[0] >> RSHIFT;
         gval = pixel[1] >> RSHIFT;
         bval = pixel[2] >> RSHIFT;
 
         if (rval < rmin) {
           rmin = rval;
-        } else if (rval > rmax) {
+        }
+        else if (rval > rmax) {
           rmax = rval;
         }
 
         if (gval < gmin) {
           gmin = gval;
-        } else if (gval > gmax) {
+        }
+        else if (gval > gmax) {
           gmax = gval;
         }
 
         if (bval < bmin) {
           bmin = bval;
-        } else if (bval > bmax) {
+        }
+        else if (bval > bmax) {
           bmax = bval;
         }
       }
index ee602b9d586eeaf1ba1a2bc75197b50b38079e93..91952d81ffab71a4306812e1948ff272d83c6dea 100644 (file)
@@ -35,20 +35,20 @@ public final class WindowWatcher implements PropertyChangeListener{
   private final Object myLock = new Object();
   private final Map<Window, WindowInfo> myWindow2Info = ContainerUtil.createWeakMap();
   /**
-   * Currenly focused window (window which has focused component). Can be {@code null} if there is no focused
+   * Currently focused window (window which has focused component). Can be {@code null} if there is no focused
    * window at all.
    */
   private Window myFocusedWindow;
   /**
    * Contains last focused window for each project.
    */
-  private final Set myFocusedWindows = new HashSet();
+  private final Set<Window> myFocusedWindows = new HashSet<>();
   @NonNls private static final String FOCUSED_WINDOW_PROPERTY = "focusedWindow";
 
   WindowWatcher() {}
 
   /**
-   * This method should get notifications abount changes of focused window.
+   * This method should get notifications about changes of focused window.
    * Only {@code focusedWindow} property is acceptable.
    * @throws IllegalArgumentException if property name isn't {@code focusedWindow}.
    */
@@ -70,8 +70,8 @@ public final class WindowWatcher implements PropertyChangeListener{
       }
       myFocusedWindow=window;
       final Project project = CommonDataKeys.PROJECT.getData(DataManager.getInstance().getDataContext(myFocusedWindow));
-      for (Iterator i = myFocusedWindows.iterator(); i.hasNext();) {
-        final Window w = (Window)i.next();
+      for (Iterator<Window> i = myFocusedWindows.iterator(); i.hasNext();) {
+        final Window w = i.next();
         final DataContext dataContext = DataManager.getInstance().getDataContext(w);
         if (project == CommonDataKeys.PROJECT.getData(dataContext)) {
           i.remove();
@@ -270,11 +270,11 @@ public final class WindowWatcher implements PropertyChangeListener{
   @Nullable
   private Window getFocusedWindowForProject(@Nullable final Project project) {
     //todo[anton,vova]: it is possible that returned wnd is not contained in myFocusedWindows; investigate
-    outer: for(Iterator i=myFocusedWindows.iterator();i.hasNext();){
-      Window window=(Window)i.next();
-      while(!window.isDisplayable()||!window.isShowing()){ // if window isn't visible then gets its first visible ancestor
-        window=window.getOwner();
-        if(window==null){
+    outer:
+    for (Window window : myFocusedWindows) {
+      while (!window.isDisplayable() || !window.isShowing()) { // if window isn't visible then gets its first visible ancestor
+        window = window.getOwner();
+        if (window == null) {
           continue outer;
         }
       }
index b7ebec77891005530ef22ae72f8b6f39d6c78da8..49d7ba9ea2fdc9b806ba3b66c3a1b51b10ee9163 100644 (file)
@@ -97,8 +97,7 @@ public class HorizontalLabeledIcon implements Icon {
 
       x += myIcon.getIconWidth() + 5;
       y += (height - getTextHeight()) / 2 + fontMetrics.getHeight() - fontMetrics.getDescent();
-      for (int i = 0; i < myStrings.length; i++) {
-        String string = myStrings[i];
+      for (String string : myStrings) {
         g.drawString(string, x, y);
         y += fontMetrics.getHeight();
       }
index 3361913b532604830156aae109314a1d94c1deef..2d3977e848fb3db68aa10257e6dab79b8a3b084d 100644 (file)
@@ -170,8 +170,8 @@ public abstract class MultilineTreeCellRenderer extends JComponent implements Ac
   public void setText(String[] lines, String prefix) {
     myLines = lines;
     myTextLength = 0;
-    for (int i = 0; i < lines.length; i++) {
-      myTextLength += lines[i].length();
+    for (String line : lines) {
+      myTextLength += line.length();
     }
     myPrefix = prefix;
 
@@ -249,13 +249,12 @@ public abstract class MultilineTreeCellRenderer extends JComponent implements Ac
     int result = 0;
     myWraps = new ArrayList();
 
-    for (int i = 0; i < myLines.length; i++) {
-      String aLine = myLines[i];
+    for (String aLine : myLines) {
       int lineFirstChar = 0;
       int lineLastChar = aLine.length() - 1;
       int currFirst = lineFirstChar;
       int printableWidth = width - myTextInsets.left - myTextInsets.right;
-      if (aLine.length() == 0) {
+      if (aLine.isEmpty()) {
         myWraps.add(aLine);
         result++;
       }
index 52254af29097a63321965ce964164f6848ccfc88..4cfa9a21ea4332f6c1a9fed77716860c59866e1b 100644 (file)
@@ -47,9 +47,8 @@ public class SelectionSaver implements TreeSelectionListener, TreeModelListener,
     TreePath pathToDelete = treeModelEvent.getTreePath();
     Object[] children = treeModelEvent.getChildren();
 
-    for (int i = 0; i < children.length; i++) {
-      Object nodeToDelete = children[i];
-      if (myCurrentSelection.contains(nodeToDelete)){
+    for (Object nodeToDelete : children) {
+      if (myCurrentSelection.contains(nodeToDelete)) {
         int deletedRow = myTree.getRowForPath(pathToDelete.pathByAddingChild(nodeToDelete));
         if (deletedRow == 0) return;
         TreePath treePath = new TreePath(myTree.getPathForRow(deletedRow - 1));
@@ -64,8 +63,7 @@ public class SelectionSaver implements TreeSelectionListener, TreeModelListener,
     myCurrentSelection = new ArrayList<>();
     TreePath[] selection = myTree.getSelectionModel().getSelectionPaths();
     if (selection == null) return;
-    for (int i = 0; i < selection.length; i++) {
-      TreePath treePath = selection[i];
+    for (TreePath treePath : selection) {
       myCurrentSelection.add((TreeNode)treePath.getLastPathComponent());
     }
   }
index c3952f509a93ab6bbb87bf5e981b5d7c401fe285..c220095e72e6842cc7db0312f7de38868bcd1353 100644 (file)
@@ -24,8 +24,7 @@ public class BridgeMethodGenerator implements CallbackGenerator {
 
   @Override
   public void generate(ClassEmitter ce, CallbackGenerator.Context context, List methods) {
-    for (Iterator it = methods.iterator(); it.hasNext();) {
-      MethodInfo method = (MethodInfo)it.next();
+    for (MethodInfo method : (List<MethodInfo>)methods) {
       final MethodInfo delegate = myCovariantInfoMap.get(method);
 
       CodeEmitter e = context.beginMethod(ce, method);
index a848a07bd7112dcfd2e9b16ebce06758a0a9cf1c..908225e592d424febd51636f51e195dcac740514 100644 (file)
@@ -55,8 +55,7 @@ public class MergeActionsTest extends TestSuite {
     protected String process(Editor editor) {
       StringBuffer buffer = new StringBuffer();
       RangeHighlighter[] allHighlighters = editor.getMarkupModel().getAllHighlighters();
-      for (int i = 0; i < allHighlighters.length; i++) {
-        RangeHighlighter highlighter = allHighlighters[i];
+      for (RangeHighlighter highlighter : allHighlighters) {
         GutterMark iconRenderer = highlighter.getGutterIconRenderer();
         if (iconRenderer != null) {
           buffer.append(iconRenderer.getTooltipText());
index 2a83d1f85d0e562a808b3414558c769b6e66da48..f85651407ec348422cd2f41debdfcaa4f0767fb6 100644 (file)
@@ -148,8 +148,7 @@ public class TreeExpandCollapseTest extends TestCase {
       treePaths[i] = myChild2Path.pathByAddingChild(child);
     }
     TreeExpandCollapse.expandAll(myTree);
-    for (int i = 0; i < treePaths.length; i++) {
-      TreePath treePath = treePaths[i];
+    for (TreePath treePath : treePaths) {
       checkExpanded(treePath);
     }
   }
index 88ac0d8a7a7df99afb3afeb246e6ffcfb6821d78..855d54571dba5ecdaf96225383e3b69d6f756ac4 100644 (file)
@@ -35,11 +35,11 @@ public class TextToIconPaintTest extends CompositeIconPaintTestHelper {
   private static final Map<ScaleContext, Integer> CTX_TO_SIZE = new HashMap<>();
 
   static {
-    for (int i=0; i<SCALES_TO_SIZE.length; i++) {
-      CTX_TO_SIZE.put(ScaleContext.create(USR_SCALE.of(SCALES_TO_SIZE[i][0]),
-                                          SYS_SCALE.of(SCALES_TO_SIZE[i][1]),
-                                          OBJ_SCALE.of(SCALES_TO_SIZE[i][2])),
-                      SCALES_TO_SIZE[i][3]);
+    for (int[] scaleData : SCALES_TO_SIZE) {
+      CTX_TO_SIZE.put(ScaleContext.create(USR_SCALE.of(scaleData[0]),
+                                          SYS_SCALE.of(scaleData[1]),
+                                          OBJ_SCALE.of(scaleData[2])),
+                      scaleData[3]);
     }
   }
 
index 3f6a278ef1dc16dce054d2c773ecf813445e28ec..f1804630c52c9b9db89c394ee0281fee3a0ce005 100644 (file)
@@ -75,8 +75,7 @@ public class FragmentListImpl implements FragmentList {
   public static ArrayList<Fragment> shift(ArrayList<Fragment> fragments, TextRange rangeShift1, TextRange rangeShift2,
                                      int startLine1, int startLine2) {
     ArrayList<Fragment> newFragments = new ArrayList<Fragment>(fragments.size());
-    for (Iterator<Fragment> iterator = fragments.iterator(); iterator.hasNext();) {
-      Fragment fragment = iterator.next();
+    for (Fragment fragment : fragments) {
       newFragments.add(fragment.shift(rangeShift1, rangeShift2, startLine1, startLine2));
     }
     return newFragments;
index 9bdd027b7a5e273ba66eba735428d215ac115efd..7382a60a7a1eeaaa1f6356e576623b481635b4a0 100644 (file)
@@ -13,19 +13,22 @@ public abstract class LineBlockDivider {
       List2D result = new List2D();
       FragmentSide currentSide = null;
       boolean isNewLineLast = true;
-      for (int i = 0; i < lineBlock.length; i++) {
-        DiffFragment fragment = lineBlock[i];
+      for (DiffFragment fragment : lineBlock) {
         if (!fragment.isOneSide()) {
           if (currentSide != null && isNewLineLast) result.newRow();
           isNewLineLast = StringUtil.endsWithChar(fragment.getText1(), '\n') && StringUtil.endsWithChar(fragment.getText2(), '\n');
           currentSide = null;
-        } else {
+        }
+        else {
           FragmentSide side = FragmentSide.chooseSide(fragment);
           if (currentSide != side) {
             if (isNewLineLast) {
               result.newRow();
               currentSide = side;
-            } else currentSide = null;
+            }
+            else {
+              currentSide = null;
+            }
           }
           isNewLineLast = StringUtil.endsWithChar(side.getText(fragment), '\n');
         }
index 0a6d905b0d376dd29e4d20441f3dea28e99c0680..7b645327af54550d6f9f62861f3b0a39e330b2e9 100644 (file)
@@ -18,6 +18,7 @@ package com.intellij.openapi.diff.impl.highlighting;
 import com.intellij.openapi.diff.ex.DiffFragment;
 
 import java.util.ArrayList;
+import java.util.Collections;
 import java.util.List;
 
 class List2D {
@@ -54,9 +55,6 @@ class List2D {
 
   public void addAll(DiffFragment[] line) {
     ensureRowExists();
-    for (int i = 0; i < line.length; i++) {
-      DiffFragment value = line[i];
-      myCurrentRow.add(value);
-    }
+    Collections.addAll(myCurrentRow, line);
   }
 }
index 69edf2d116a2fee6b7e978ec063575b464580820..6af62322f9b4a7afe70e445fb76c34e80134de95 100644 (file)
@@ -37,8 +37,7 @@ public class Util {
 
   static {
     char[] delimiters = DELIMITERS.toCharArray();
-    for (int i = 0; i < delimiters.length; i++) {
-      char delimiter = delimiters[i];
+    for (char delimiter : delimiters) {
       DELIMITERS_SET.add(delimiter);
     }
   }
@@ -101,8 +100,7 @@ public class Util {
   @NotNull
   public static DiffFragment[][] splitByUnchangedLines(@NotNull DiffFragment[] fragments) {
     List2D result = new List2D();
-    for (int i = 0; i < fragments.length; i++) {
-      DiffFragment fragment = fragments[i];
+    for (DiffFragment fragment : fragments) {
       if (!fragment.isEqual()) {
         result.add(fragment);
         continue;
@@ -130,8 +128,9 @@ public class Util {
         int length1 = text1.length();
         int length2 = text2.length();
         if (length1 == 0 || length2 == 0) {
-          if (length1 != 0 || length2 != 0)
+          if (length1 != 0 || length2 != 0) {
             result.add(DiffFragment.unchanged(text1, text2));
+          }
           break;
         }
       }
@@ -254,15 +253,16 @@ public class Util {
 
   @Nullable
   private static DiffString notEmptyContent(@NotNull DiffString string) {
-    return string.length() > 0 ? string : null;
+    return !string.isEmpty() ? string : null;
   }
 
   @NotNull
   public static DiffFragment[][] uniteFormattingOnly(@NotNull DiffFragment[][] lines) {
     List2D result = new List2D();
-    for (int i = 0; i < lines.length; i++) {
-      DiffFragment[] line = lines[i];
-      if (!areEqual(line) && areEqualOrFormatting(line)) result.addAll(line);
+    for (DiffFragment[] line : lines) {
+      if (!areEqual(line) && areEqualOrFormatting(line)) {
+        result.addAll(line);
+      }
       else {
         result.newRow();
         result.addAll(line);
@@ -273,10 +273,9 @@ public class Util {
   }
 
   private static boolean areEqualOrFormatting(@NotNull DiffFragment[] fragments) {
-    for (int i = 0; i < fragments.length; i++) {
-      DiffFragment fragment = fragments[i];
+    for (DiffFragment fragment : fragments) {
       if (fragment.isEqual()) continue;
-      for (int side = 0;  side < 2; side++) {
+      for (int side = 0; side < 2; side++) {
         DiffString text = FragmentSide.fromIndex(side).getText(fragment);
         if (text == null || text.isEmptyOrSpaces()) continue;
         return false;
@@ -286,8 +285,7 @@ public class Util {
   }
 
   private static boolean areEqual(@NotNull DiffFragment[] fragments) {
-    for (int i = 0; i < fragments.length; i++) {
-      DiffFragment fragment = fragments[i];
+    for (DiffFragment fragment : fragments) {
       if (!fragment.isEqual()) return false;
     }
     return true;
@@ -318,8 +316,7 @@ public class Util {
     if (nullCount == 0) return fragments;
     DiffFragment[] result = new DiffFragment[fragments.length - nullCount];
     int dstIndex = 0;
-    for (int i = 0; i < fragments.length; i++) {
-      DiffFragment fragment = fragments[i];
+    for (DiffFragment fragment : fragments) {
       if (fragment == null) continue;
       result[dstIndex] = fragment;
       dstIndex++;
index a5f6272065a2509526405246607741fd1a23f66c..6b033d597349439fef76cf0a0eacdbb0f8ec5055 100644 (file)
@@ -79,8 +79,7 @@ public class ByWord implements DiffPolicy {
 
   private static int countNotWhitespaces(@NotNull Word[] words) {
     int counter = 0;
-    for (int i = 0; i < words.length; i++) {
-      Word word = words[i];
+    for (Word word : words) {
       if (!word.isWhitespace()) counter++;
     }
     return counter;
@@ -197,8 +196,7 @@ public class ByWord implements DiffPolicy {
     public Version getVersion2() { return myVersion2; }
 
     private void addAll(@NotNull DiffFragment[] fragments) {
-      for (int i = 0; i < fragments.length; i++) {
-        DiffFragment fragment = fragments[i];
+      for (DiffFragment fragment : fragments) {
         add(fragment);
       }
     }
index ffb4057f8698982a9d758bec550703f5e29ceb7a..060f540f824b9c755c40ed1098786c2fc2ce1c46 100644 (file)
@@ -169,8 +169,8 @@ public interface DiffCorrection {
 
   class FragmentsCollector extends BaseFragmentRunner<FragmentsCollector> {
     public void addAll(DiffFragment[] fragments) {
-      for (int i = 0; i < fragments.length; i++) {
-        add(fragments[i]);
+      for (DiffFragment fragment : fragments) {
+        add(fragment);
       }
     }
   }
index bf21f1066ad2cc839ea172cea088fce284dec3b4..3dee295630f5c1ce25c2bd5141f642a3fabd7432 100644 (file)
@@ -56,10 +56,11 @@ class UniteSameType implements DiffCorrection {
     DiffCorrection.FragmentsCollector collector = new DiffCorrection.FragmentsCollector();
 //    DiffFragment previous = fragments[0];
     DiffFragment previous = null;
-    for (int i = 0; i < fragments.length; i++) {
-      DiffFragment fragment = fragments[i];
+    for (DiffFragment fragment : fragments) {
       if (fragment.isOneSide()) {
-        if (previous == null) previous = fragment;
+        if (previous == null) {
+          previous = fragment;
+        }
         else {
           FragmentSide side = FragmentSide.chooseSide(fragment);
           DiffString previousText = side.getText(previous);
@@ -67,7 +68,8 @@ class UniteSameType implements DiffCorrection {
           previous = side.createFragment(DiffString.concatenateNullable(previousText, side.getText(fragment)),
                                          side.getOtherText(previous), true);
         }
-      } else {
+      }
+      else {
         if (previous != null) collector.add(previous);
         previous = null;
         collector.add(fragment);
index 084b80699c4b31b34a70bece357e4cf81b903863..93da9989285b945e736e96d7905f13b44e17ea27 100644 (file)
@@ -42,9 +42,9 @@ public class GeometryUtil implements SwingConstants {
     bounds[2] = getBottomOf(aRectangle);
     bounds[3] = getLeftOf(aRectangle);
 
-    for (int i = 0; i < bounds.length; i++) {
-      if (bounds[i].intersectsLine(aSegment)) {
-        return getIntersectionPoint(aSegment, bounds[i]);
+    for (Line2D bound : bounds) {
+      if (bound.intersectsLine(aSegment)) {
+        return getIntersectionPoint(aSegment, bound);
       }
     }
 
@@ -98,8 +98,8 @@ public class GeometryUtil implements SwingConstants {
     double bSecond = aSecond.getY1() - kSecond * aSecond.getX1();
 
 
-    double xIntersection = -100000000;
-    double yIntersection = -100000000;
+    double xIntersection;
+    double yIntersection;
 
 
     double deltaK = (kFirst - kSecond);
index b5315f15da7e24f9d56f86d5c71ba8772988eaef..6dd9bed5ee6ee3e7c687339fc64d2a2149a6dbe7 100644 (file)
@@ -17,7 +17,6 @@ package com.intellij.util.ui;
 
 import java.io.File;
 import java.util.ArrayList;
-import java.util.Iterator;
 import java.util.List;
 
 /**
@@ -63,8 +62,8 @@ public class SplitBySeparatorPolicy extends FilePathSplittingPolicy {
 
     StringBuilder result = new StringBuilder();
 
-    for (Iterator<String> iterator = begin.iterator(); iterator.hasNext();) {
-      result.append(iterator.next());
+    for (String line : begin) {
+      result.append(line);
     }
 
     result.append("...");
@@ -76,7 +75,7 @@ public class SplitBySeparatorPolicy extends FilePathSplittingPolicy {
     return result.toString();
   }
 
-  private ArrayList<String> getComponents(File file) {
+  private static ArrayList<String> getComponents(File file) {
     ArrayList<String> result = new ArrayList<String>();
     File current = file;
     while (current != null) {
@@ -87,7 +86,7 @@ public class SplitBySeparatorPolicy extends FilePathSplittingPolicy {
     return result;
   }
 
-  private String getFileName(File current) {
+  private static String getFileName(File current) {
     String result = current.getName();
     if (!result.isEmpty()) return result;
     String path = current.getPath();
index d9a154f4942cd27479991ca399f662af46d8f883..dd5cb3d8e254459403b532f1766e4d138d44555b 100644 (file)
@@ -104,15 +104,15 @@ public class UniqueLCSTest extends TestCase {
 
   private static void checkMaxSequence(int[] sequence, int[] expected) {
     int max = 0;
-    for (int i = 0; i < sequence.length; i++) {
-      max = Math.max(sequence[i] + 1, max);
+    for (int aSequence : sequence) {
+      max = Math.max(aSequence + 1, max);
     }
 
     int[] first = new int[sequence.length];
     int[] second = new int[max + 2];
 
     for (int i = 0; i < sequence.length; i++) {
-      assertEquals("Elements in sequence should be unique", second[sequence[i]], 0);
+      assertEquals("Elements in sequence should be unique", 0, second[sequence[i]]);
       first[i] = i + 1;
       second[sequence[i]] = i + 1;
     }
index e5051a71badcfcd47e7294acad615d3b3d69cf5f..600e3d04d5fe612a0358cc3706178db0c1883ec2 100644 (file)
@@ -98,8 +98,7 @@ public class ReplaceFileConfirmationDialog {
 
     if (files == null) return result;
 
-    for (int i = 0; i < files.length; i++) {
-      VirtualFile file = files[i];
+    for (VirtualFile file : files) {
       if (myProgressIndicator != null) {
         myProgressIndicator.setText(VcsBundle.message("progress.text.searching.for.modified.files"));
         myProgressIndicator.setText2(file.getPresentableUrl());
index 82db3661445f8f9c1658d151bf08db6fa7274d33..b6cdd75c284014f638ef2facbbe4a1f43d49f555 100644 (file)
@@ -83,8 +83,7 @@ public class CustomChangelistTodosTreeBuilder extends TodoTreeBuilder {
             }
           }
         }
-        for (Iterator<PsiFile> iterator = myIncludedFiles.iterator(); iterator.hasNext(); ) {
-          final PsiFile next = iterator.next();
+        for (final PsiFile next : myIncludedFiles) {
           final Change change = allChanges.get(next.getVirtualFile());
           if (change != null) {
             changes.add(change);
@@ -166,7 +165,7 @@ public class CustomChangelistTodosTreeBuilder extends TodoTreeBuilder {
       }
     }
     final Collection<TodoItem> todoItems = myMap.get(file);
-    return todoItems == null || todoItems.isEmpty() ? EMPTY_ITEMS : todoItems.toArray(new TodoItem[0]);
+    return todoItems.isEmpty() ? EMPTY_ITEMS : todoItems.toArray(new TodoItem[0]);
   }
 
   @NotNull
@@ -180,9 +179,8 @@ public class CustomChangelistTodosTreeBuilder extends TodoTreeBuilder {
     Set<VirtualFile> files = new HashSet<>();
     TodoTreeStructure treeStructure=getTodoTreeStructure();
     PsiFile[] psiFiles= myPsiTodoSearchHelper.findFilesWithTodoItems();
-    for(int i=0;i<psiFiles.length;i++){
-      PsiFile psiFile=psiFiles[i];
-      if(myPsiTodoSearchHelper.getTodoItemsCount(psiFile) > 0 && treeStructure.accept(psiFile)){
+    for (PsiFile psiFile : psiFiles) {
+      if (myPsiTodoSearchHelper.getTodoItemsCount(psiFile) > 0 && treeStructure.accept(psiFile)) {
         files.add(psiFile.getVirtualFile());
       }
     }
index 4f5d1777e2faf130226e9ac2479964c8c7df4eb1..b6011d4b81994ca60886f6fb171beca1f308f6d7 100644 (file)
@@ -334,8 +334,7 @@ public class GenericPatchApplier {
   // applies in a way that patch _can_ be solved manually even in the case of total mismatch
   public void trySolveSomehow() {
     assert !myNotExact.isEmpty();
-    for (Iterator<SplitHunk> iterator = myNotExact.iterator(); iterator.hasNext(); ) {
-      final SplitHunk hunk = iterator.next();
+    for (final SplitHunk hunk : myNotExact) {
       hunk.cutSameTail();
       if (!testForPartialContextMatch(hunk, new LongTryMismatchSolver(hunk), ourMaxWalk, null)) {
         if (complementIfShort(hunk)) {
@@ -948,7 +947,7 @@ public class GenericPatchApplier {
     final int offset = splitHunk.getContextBefore().size();
     final List<BeforeAfter<List<String>>> steps = splitHunk.getPatchSteps();
     if (splitHunk.isInsertion()) {
-      final boolean emptyFile = myLines.isEmpty() || myLines.size() == 1 && myLines.get(0).trim().length() == 0;
+      final boolean emptyFile = myLines.isEmpty() || myLines.size() == 1 && myLines.get(0).trim().isEmpty();
       if (emptyFile) {
         myNotBound.add(splitHunk);
         processAppliedInfoForUnApplied(splitHunk);
@@ -992,15 +991,12 @@ public class GenericPatchApplier {
     private int myOffsetIdxInHunk;
     // if we set index in hunk != 0, then we will check only one side
     private Boolean myBeforeSide;
-    private boolean myIsInBefore;
-    private final int myIdxInHunk;
     private final BeforeAfter<List<String>> myBeforeAfter;
 
     private FragmentMatcher(int idx, BeforeAfter<List<String>> beforeAfter) {
       myOffsetIdxInHunk = 0;
       myIdx = idx;
       myBeforeAfter = beforeAfter;
-      myIdxInHunk = 0;
       //myBeforeSide = true;
     }
     
@@ -1008,9 +1004,9 @@ public class GenericPatchApplier {
       myOffsetIdxInHunk = startInHunk;
       myBeforeSide = beforeSide;
       if (myBeforeSide) {
-        assert myBeforeAfter.getBefore().size() > myOffsetIdxInHunk || (myOffsetIdxInHunk == 0 && myBeforeAfter.getBefore().size() == 0);
+        assert myBeforeAfter.getBefore().size() > myOffsetIdxInHunk || (myOffsetIdxInHunk == 0 && myBeforeAfter.getBefore().isEmpty());
       } else {
-        assert myBeforeAfter.getAfter().size() > myOffsetIdxInHunk || (myOffsetIdxInHunk == 0 && myBeforeAfter.getAfter().size() == 0);
+        assert myBeforeAfter.getAfter().size() > myOffsetIdxInHunk || (myOffsetIdxInHunk == 0 && myBeforeAfter.getAfter().isEmpty());
       }
     }
 
@@ -1231,20 +1227,20 @@ public class GenericPatchApplier {
       int i = startI;
       for (; i < lines.size(); i++) {
         final PatchLine patchLine = lines.get(i);
-        if (! PatchLine.Type.CONTEXT.equals(patchLine.getType())) break;
+        if (PatchLine.Type.CONTEXT != patchLine.getType()) break;
         contextBefore.add(patchLine.getText());
       }
 
-      final boolean addFirst = i < lines.size() && PatchLine.Type.ADD.equals(lines.get(i).getType());
+      final boolean addFirst = i < lines.size() && PatchLine.Type.ADD == lines.get(i).getType();
       List<String> before = new ArrayList<>();
       List<String> after = new ArrayList<>();
       for (; i < lines.size(); i++) {
         final PatchLine patchLine = lines.get(i);
         final PatchLine.Type type = patchLine.getType();
-        if (PatchLine.Type.CONTEXT.equals(type)) {
+        if (PatchLine.Type.CONTEXT == type) {
           break;
         }
-        if (PatchLine.Type.ADD.equals(type)) {
+        if (PatchLine.Type.ADD == type) {
           if (addFirst && ! before.isEmpty()) {
             // new piece
             steps.add(new BeforeAfter<>(before, after));
@@ -1252,7 +1248,7 @@ public class GenericPatchApplier {
             after = new ArrayList<>();
           }
           after.add(patchLine.getText());
-        } else if (PatchLine.Type.REMOVE.equals(type)) {
+        } else if (PatchLine.Type.REMOVE == type) {
           if (! addFirst && ! after.isEmpty()) {
             // new piece
             steps.add(new BeforeAfter<>(before, after));
@@ -1268,7 +1264,7 @@ public class GenericPatchApplier {
 
       for (; i < lines.size(); i++) {
         final PatchLine patchLine = lines.get(i);
-        if (! PatchLine.Type.CONTEXT.equals(patchLine.getType())) {
+        if (PatchLine.Type.CONTEXT != patchLine.getType()) {
           return i;
         }
         contextAfter.add(patchLine.getText());
index 38e307d31f273d66497c86dec44579c076e7287c..babf42ccb62ee5d873d64b3ee5d7916dcc57678a 100644 (file)
@@ -23,6 +23,7 @@ import junit.framework.TestCase;
 
 import java.util.Arrays;
 import java.util.List;
+import java.util.stream.Collectors;
 
 public class RangeBuilderTest extends TestCase {
   public void testIdenticalContents() {
@@ -133,11 +134,6 @@ public class RangeBuilderTest extends TestCase {
   }
 
   private static String createContentOn(String[] content) {
-    StringBuilder result = new StringBuilder();
-    for (int i = 0; i < content.length; i++) {
-      result.append(content[i]);
-      result.append('\n');
-    }
-    return result.toString();
+    return Arrays.stream(content).map(line -> line + '\n').collect(Collectors.joining());
   }
 }
index 3c1873d32a00f74684af7daf6bfc647bb7d0241c..cdedb69fedd5d35129b10bb4e69d87de37c7f50f 100644 (file)
@@ -91,8 +91,7 @@ public class AntClasspathEditorPanel extends JPanel {
     }
 
     public void enableButtons(boolean enable) {
-      for (Iterator<ListUtil.Updatable> iterator = myUpdatables.iterator(); iterator.hasNext();) {
-        ListUtil.Updatable updatable = iterator.next();
+      for (ListUtil.Updatable updatable : myUpdatables) {
         updatable.enable(enable);
       }
     }
index f0624372d8f6c59ca40003b34e73dbdb95dd35a6..f3300590b64f4c83b6d974734a5b61b6dc959715 100644 (file)
@@ -38,8 +38,8 @@ public abstract class AbstractWatchAction extends AbstractActionFromEditGroup {
     saveWatch(configuration, watch);
     WatchOperation watchOperation = new WatchOperation(getWatchOperation(), watch);
     VirtualFile[] files = context.getSelectedFiles();
-    for (int i = 0; i < files.length; i++) {
-      watchOperation.addFile(files[i]);
+    for (VirtualFile file : files) {
+      watchOperation.addFile(file);
     }
     return new CommandCvsHandler(getTitle(context), watchOperation);
   }
index 613b5173df67e84a54b7265e2366dc7bcfb4e639..3b9a4802a522623ad4bc267c49bee653c19490f4 100644 (file)
@@ -95,9 +95,7 @@ public abstract class ActionOnSelectedElement extends AbstractAction {
     @Override
     public boolean isPerformedOn(CvsContext context) {
       VirtualFile[] selectedFiles = context.getSelectedFiles();
-      if (selectedFiles == null) return false;
-      for (int i = 0; i < selectedFiles.length; i++) {
-        VirtualFile selectedFile = selectedFiles[i];
+      for (VirtualFile selectedFile : selectedFiles) {
         if (CvsStatusProvider.getStatus(selectedFile) == FileStatus.NOT_CHANGED) {
           return documentIsModified(selectedFile);
         }
@@ -110,7 +108,6 @@ public abstract class ActionOnSelectedElement extends AbstractAction {
     @Override
     public boolean isPerformedOn(CvsContext context) {
       VirtualFile[] selectedFiles = context.getSelectedFiles();
-      if (selectedFiles == null) return false;
       final CvsEntriesManager entriesManager = CvsEntriesManager.getInstance();
       for (VirtualFile selectedFile : selectedFiles) {
         if (entriesManager.fileIsIgnored(selectedFile)) return false;
@@ -123,7 +120,6 @@ public abstract class ActionOnSelectedElement extends AbstractAction {
     @Override
     public boolean isPerformedOn(CvsContext context) {
       VirtualFile[] selectedFiles = context.getSelectedFiles();
-      if (selectedFiles == null) return false;
       for (VirtualFile selectedFile : selectedFiles) {
         if (!CvsUtil.fileIsLocallyAdded(selectedFile)) return false;
       }
@@ -133,7 +129,7 @@ public abstract class ActionOnSelectedElement extends AbstractAction {
 
 
   private static boolean documentIsModified(final VirtualFile file) {
-    final boolean[] result = new boolean[]{false};
+    final boolean[] result = {false};
     ApplicationManager.getApplication().runReadAction(() -> {
       result[0] = FileDocumentManager.getInstance().isFileModified(file);
     });
index 1fce3982b0077a5e691d95ab9506c384a2d499fc..b80346b49ed7c605bebfbd3cc5bae659acbd4644 100644 (file)
@@ -30,8 +30,7 @@ class SelectedFiles extends AnyProcessedFiles {
   private final Collection<VirtualFile> myFiles = new ArrayList<>();
 
   public SelectedFiles(FilePath[] files) {
-    for (int i = 0; i < files.length; i++) {
-      FilePath file = files[i];
+    for (FilePath file : files) {
       VirtualFile virtualFile = file.getVirtualFile();
       if (virtualFile != null) {
         myFiles.add(virtualFile);
index be91caa61a439e96f7409dc86e54b3dd62fee931..b5de1affa8e53382e28e636a6b60dd87f8efd821 100644 (file)
@@ -31,8 +31,8 @@ import java.util.List;
 public class WatchersOperation extends CvsOperationOnFiles{
   private final List<WatcherInfo> myWatchers = new ArrayList<>();
   public WatchersOperation(VirtualFile[] files){
-    for (int i = 0; i < files.length; i++) {
-      addFile(files[i]);
+    for (VirtualFile file : files) {
+      addFile(file);
     }
 
   }
index bbcced9be7a4d5236927ac954c01f039c276421c..eb4c83860606bcdddc2b621a365050ce8dff9491 100644 (file)
@@ -44,9 +44,8 @@ public class AdminReaderForLightFiles implements IAdminReader{
   }
 
   private Collection collectEntriesForPath(File parent) {
-    HashSet result = new HashSet();
-    for (Iterator iterator = myFileToEntryMap.keySet().iterator(); iterator.hasNext();) {
-      File file = (File) iterator.next();
+    HashSet<Entry> result = new HashSet<>();
+    for (File file : myFileToEntryMap.keySet()) {
       if (file.getParentFile().equals(parent)) result.add(myFileToEntryMap.get(file));
     }
     return result;
@@ -70,8 +69,7 @@ public class AdminReaderForLightFiles implements IAdminReader{
   }
 
   private Entry getEntryForPath(File requested) {
-    for (Iterator iterator = myFileToEntryMap.keySet().iterator(); iterator.hasNext();) {
-      File file = (File) iterator.next();
+    for (File file : myFileToEntryMap.keySet()) {
       if (file.equals(requested)) return myFileToEntryMap.get(file);
     }
     return null;
index ebcc0412719feea16da92588840262b6dff521f3..2106d52b320bbb2600061c15423304771b06d847 100644 (file)
@@ -150,8 +150,8 @@ public final class RequestProcessor implements IRequestProcessor {
     if (envVariables == null) {
       return;
     }
-    for (Iterator iterator = envVariables.keySet().iterator(); iterator.hasNext();) {
-      String varName = (String)iterator.next();
+    for (Object o : envVariables.keySet()) {
+      String varName = (String)o;
       String varValue = (String)envVariables.get(varName);
       sendRequest(new SetRequest(varName, varValue), connectionStreams);
     }
@@ -277,8 +277,8 @@ public final class RequestProcessor implements IRequestProcessor {
 
   private void sendRequests(Requests requests, IConnectionStreams connectionStreams, IRequestsProgressHandler communicationProgressHandler)
     throws CommandAbortedException, IOException {
-    for (Iterator it = requests.getRequests().iterator(); it.hasNext();) {
-      final IRequest request = (IRequest)it.next();
+    for (Object o : requests.getRequests()) {
+      final IRequest request = (IRequest)o;
 
       sendRequest(request, connectionStreams);
 
index 2d9260a4dd63c83c171924ed9f87100673296dee..b92d8608feeba50ec14400967389aeeecc42828b 100644 (file)
@@ -17,7 +17,6 @@ import org.netbeans.lib.cvsclient.file.FileObject;
 import org.netbeans.lib.cvsclient.util.BugLog;
 
 import java.util.ArrayList;
-import java.util.Iterator;
 import java.util.List;
 
 /**
@@ -28,7 +27,7 @@ final class CvsFiles
 
        // Fields =================================================================
 
-       private final List cvsFiles = new ArrayList();
+       private final List<CvsFile> cvsFiles = new ArrayList<>();
        private CvsFile currentCvsDirectory;
 
        // Setup ==================================================================
@@ -42,8 +41,7 @@ final class CvsFiles
         public void visit(ICvsFilesVisitor visitor) {
                BugLog.getInstance().assertNotNull(visitor);
 
-               for (Iterator it = cvsFiles.iterator(); it.hasNext();) {
-                       final CvsFile cvsFile = (CvsFile)it.next();
+               for (CvsFile cvsFile : cvsFiles) {
                        if (cvsFile.isDirectory()) {
                                visitor.handleDirectory((DirectoryObject)cvsFile.getFileObject());
                        }
index 9ca716820cd6e330a528da98ce7063b6c62566c2..e26709c0be5b632fd3635ba065c98b3d3c8f22be 100644 (file)
@@ -27,7 +27,6 @@ import org.netbeans.lib.cvsclient.util.BugLog;
 import java.io.FileNotFoundException;
 import java.io.IOException;
 import java.util.ArrayList;
-import java.util.Iterator;
 import java.util.List;
 
 /**
@@ -38,7 +37,7 @@ public final class DirectoryPruner
 
        // Fields =================================================================
 
-       private final List directoriesToScan = new ArrayList();
+       private final List<DirectoryObject> directoriesToScan = new ArrayList<>();
        private final IClientEnvironment clientEnvironment;
 
        // Setup ==================================================================
@@ -80,8 +79,8 @@ public final class DirectoryPruner
         * Remove any directories that don't contain any files
         */
        public final void pruneEmptyDirectories() throws IOException {
-               while (directoriesToScan.size() > 0) {
-                       final DirectoryObject directoryObject = (DirectoryObject)directoriesToScan.remove(0);
+               while (!directoriesToScan.isEmpty()) {
+                       final DirectoryObject directoryObject = directoriesToScan.remove(0);
                        pruneEmptyDirectory(directoryObject);
                }
        }
@@ -100,11 +99,11 @@ public final class DirectoryPruner
                        return true;
                }
 
-               final List fileNames = new ArrayList();
-               final List directoryNames = new ArrayList();
+               final List<String> fileNames = new ArrayList<>();
+               final List<String> directoryNames = new ArrayList<>();
                localFileReader.listFilesAndDirectories(directoryObject, fileNames, directoryNames, cvsFileSystem);
 
-               if (fileNames.size() > 0) {
+               if (!fileNames.isEmpty()) {
                        return false;
                }
 
@@ -112,9 +111,7 @@ public final class DirectoryPruner
                        return false;
                }
 
-               for (Iterator it = directoryNames.iterator(); it.hasNext();) {
-                       final String directoryName = (String)it.next();
-
+               for (String directoryName : directoryNames) {
                        if (!pruneEmptyDirectory(DirectoryObject.createInstance(directoryObject, directoryName))) {
                                return false;
                        }
@@ -135,9 +132,8 @@ public final class DirectoryPruner
                return false;
        }
 
-       private boolean hasFileEntry(DirectoryObject directoryObject, ICvsFileSystem cvsFileSystem, IAdminReader adminReader) throws IOException {
-               for (Iterator it = adminReader.getEntries(directoryObject, cvsFileSystem).iterator(); it.hasNext();) {
-                       final Entry entry = (Entry)it.next();
+       private static boolean hasFileEntry(DirectoryObject directoryObject, ICvsFileSystem cvsFileSystem, IAdminReader adminReader) throws IOException {
+               for (final Entry entry : adminReader.getEntries(directoryObject, cvsFileSystem)) {
                        if (!entry.isDirectory()) {
                                return true;
                        }
index 4cef2cb35fb799dc547e9fcf951d2b998010f94e..9f2b4267b02974b15efa5693e6d39fb2498edfac 100644 (file)
@@ -271,8 +271,7 @@ public final class LogInformation {
                if (symbolicNames == null) {
                        createSymNames();
                }
-               for (Iterator it = symbolicNames.iterator(); it.hasNext();) {
-                       final SymbolicName item = (SymbolicName)it.next();
+               for (final SymbolicName item : symbolicNames) {
                        if (item.getName().equals(symName)) {
                                return item;
                        }
index f089435cd0f5748798cd040f148d2def0415bfc2..82809f8166ec97ec762849153d4da11c0d0b6bb5 100644 (file)
@@ -17,7 +17,6 @@ package org.netbeans.lib.cvsclient.command.status;
 import org.netbeans.lib.cvsclient.file.FileStatus;
 
 import java.io.File;
-import java.util.Iterator;
 import java.util.LinkedList;
 import java.util.List;
 
@@ -43,7 +42,7 @@ public final class StatusInformation {
        /**
         * Hold key pairs of existing tags.
         */
-       private List tags;
+       private List<SymName> tags;
 
        private StringBuffer symNamesBuffer;
 
@@ -154,7 +153,7 @@ public final class StatusInformation {
        }
 
        private void createSymNames() {
-               tags = new LinkedList();
+               tags = new LinkedList<>();
 
                if (symNamesBuffer == null) {
                        return;
@@ -201,10 +200,9 @@ public final class StatusInformation {
                        createSymNames();
                }
 
-               final List list = new LinkedList();
+               final List<SymName> list = new LinkedList<>();
 
-               for (Iterator it = tags.iterator(); it.hasNext();) {
-                       final StatusInformation.SymName item = (StatusInformation.SymName)it.next();
+               for (final SymName item : tags) {
                        if (item.getRevision().equals(revNumber)) {
                                list.add(item);
                        }
@@ -221,8 +219,7 @@ public final class StatusInformation {
                        createSymNames();
                }
 
-               for (Iterator it = tags.iterator(); it.hasNext();) {
-                       final StatusInformation.SymName item = (StatusInformation.SymName)it.next();
+               for (final SymName item : tags) {
                        if (item.getTag().equals(tagName)) {
                                return item;
                        }
@@ -250,12 +247,12 @@ public final class StatusInformation {
                buf.append(stickyOptions);
                buf.append("\nSticky tag: ");
                buf.append(stickyTag);
-               if (tags != null && tags.size() > 0) {
+               if (tags != null && !tags.isEmpty()) {
                        // we are having some tags to print
                        buf.append("\nExisting Tags:");
-                       for (Iterator it = tags.iterator(); it.hasNext();) {
+                       for (SymName tag : tags) {
                                buf.append("\n  ");
-                               buf.append(it.next().toString());
+                               buf.append(tag.toString());
                        }
                }
                return buf.toString();
index 536187f0b1d43b347e6b00bfb6dad5f908a6c663..3d197174461678a228c69671e8a711f477c5f6fb 100644 (file)
@@ -70,9 +70,9 @@ public final class WatchCommand extends AbstractCommand {
                addFileRequests(cvsFiles, requests, clientEnvironment);
                if (getWatchMode().isWatchOptionAllowed()) {
                        final String[] arguments = getWatchNotNull().getArguments();
-                       for (int i = 0; i < arguments.length; i++) {
+                       for (String argument : arguments) {
                                requests.addArgumentRequest("-a");
-                               requests.addArgumentRequest(arguments[i]);
+                               requests.addArgumentRequest(argument);
                        }
                }
                requests.addLocalPathDirectoryRequest();
index 851d406de7cdd40c606047040d08380b22900df7..1fc8f69c290cccc86b4549135427d64191a8e596 100644 (file)
@@ -117,7 +117,7 @@ public final class EventManager
         public void notifyTerminationListeners(boolean error) {
                 final ITerminationListener[] copiedListeners;
                 synchronized (this) {
-                        if (terminationListeners.size() == 0) {
+                  if (terminationListeners.isEmpty()) {
                                 return;
                         }
 
@@ -125,16 +125,16 @@ public final class EventManager
                         terminationListeners.toArray(copiedListeners);
                 }
 
-                for (int i = 0; i < copiedListeners.length; i++) {
-                        copiedListeners[i].commandTerminated(error);
-                }
+          for (ITerminationListener copiedListener : copiedListeners) {
+            copiedListener.commandTerminated(error);
+          }
         }
 
         @Override
         public void notifyMessageListeners(byte[] message, boolean error, boolean tagged) {
                 final IMessageListener[] copiedListeners;
                 synchronized (this) {
-                        if (messageListener.size() == 0) {
+                  if (messageListener.isEmpty()) {
                                 return;
                         }
 
@@ -144,10 +144,10 @@ public final class EventManager
           try {
 
                 String stringMessage = new String(message, myCharset);
-                for (int i = 0; i < copiedListeners.length; i++) {
+            for (IMessageListener copiedListener : copiedListeners) {
 
-                    copiedListeners[i].messageSent(stringMessage, message, error, tagged);
-                }
+              copiedListener.messageSent(stringMessage, message, error, tagged);
+            }
           }
           catch (UnsupportedEncodingException e) {
             //
@@ -159,7 +159,7 @@ public final class EventManager
         public void notifyModuleExpansionListeners(String module) {
                 final IModuleExpansionListener[] copiedListeners;
                 synchronized (this) {
-                        if (moduleExpansionListeners.size() == 0) {
+                  if (moduleExpansionListeners.isEmpty()) {
                                 return;
                         }
 
@@ -167,16 +167,16 @@ public final class EventManager
                         moduleExpansionListeners.toArray(copiedListeners);
                 }
 
-                for (int i = 0; i < copiedListeners.length; i++) {
-                        copiedListeners[i].moduleExpanded(module);
-                }
+          for (IModuleExpansionListener copiedListener : copiedListeners) {
+            copiedListener.moduleExpanded(module);
+          }
         }
 
         @Override
         public void notifyFileInfoListeners(Object fileInfoContainer) {
                 final IFileInfoListener[] copiedListeners;
                 synchronized (this) {
-                        if (fileInfoListeners.size() == 0) {
+                  if (fileInfoListeners.isEmpty()) {
                                 return;
                         }
 
@@ -184,16 +184,16 @@ public final class EventManager
                         fileInfoListeners.toArray(copiedListeners);
                 }
 
-                for (int i = 0; i < copiedListeners.length; i++) {
-                        copiedListeners[i].fileInfoGenerated(fileInfoContainer);
-                }
+          for (IFileInfoListener copiedListener : copiedListeners) {
+            copiedListener.fileInfoGenerated(fileInfoContainer);
+          }
         }
 
         @Override
         public void notifyFileInfoListeners(byte[] bytes) {
           final IMessageListener[] copiedListeners;
           synchronized (this) {
-                  if (messageListener.size() == 0) {
+            if (messageListener.isEmpty()) {
                           return;
                   }
 
@@ -201,8 +201,8 @@ public final class EventManager
                   messageListener.toArray(copiedListeners);
           }
 
-          for (int i = 0; i < copiedListeners.length; i++) {
-                  copiedListeners[i].binaryMessageSent(bytes);
+          for (IMessageListener copiedListener : copiedListeners) {
+            copiedListener.binaryMessageSent(bytes);
           }
 
 
@@ -212,7 +212,7 @@ public final class EventManager
   public void notifyEntryListeners(FileObject fileObject, Entry entry) {
                 final IEntryListener[] copiedListeners;
                 synchronized (this) {
-                        if (entryListeners.size() == 0) {
+                  if (entryListeners.isEmpty()) {
                                 return;
                         }
 
@@ -220,16 +220,16 @@ public final class EventManager
                         entryListeners.toArray(copiedListeners);
                 }
 
-                for (int i = 0; i < copiedListeners.length; i++) {
-                        copiedListeners[i].gotEntry(fileObject, entry);
-                }
+    for (IEntryListener copiedListener : copiedListeners) {
+      copiedListener.gotEntry(fileObject, entry);
+    }
         }
 
         @Override
         public void notifyDirectoryListeners(DirectoryObject directoryObject, boolean setStatic) {
-                for (Iterator it = directoryListeners.iterator(); it.hasNext();) {
-                        final IDirectoryListener directoryListener = (IDirectoryListener)it.next();
-                        directoryListener.processingDirectory(directoryObject);
-                }
+          for (Object directoryListener1 : directoryListeners) {
+            final IDirectoryListener directoryListener = (IDirectoryListener)directoryListener1;
+            directoryListener.processingDirectory(directoryObject);
+          }
         }
 }
\ No newline at end of file
index b798179c078344a31599fee976e574bcfb1fabd2..e2343ba53a28c79f5f13bd563e4b37bebcf3cde1 100644 (file)
@@ -227,8 +227,7 @@ public final class AdminWriter implements IAdminWriter {
       return;
     }
 
-    for (int i = 0; i < files.length; i++) {
-      final File file = files[i];
+    for (final File file : files) {
       if (file.isDirectory()) {
         deleteDirectoryRecursively(file);
       }
index 32effc8022c70b719fcf4e117c84fc26f4f45914..ecc4bad511e05817b239b2beda350654dc67963d 100644 (file)
@@ -142,8 +142,7 @@ public final class LocalFileReader
                        return;
                }
 
-               for (int i = 0; i < filesAndDirectories.length; i++) {
-                       final File fileOrDirectory = filesAndDirectories[i];
+               for (final File fileOrDirectory : filesAndDirectories) {
                        final String name = fileOrDirectory.getName();
                        if (name.equals(CVS_DIR)) {
                                continue;
index fa2c1950e3fdfd63207e8ca3b770967f9145216c..34175a6b5aeb9ae92c6c3e429c3f670355953354 100644 (file)
@@ -15,7 +15,6 @@ package org.netbeans.lib.cvsclient.util;
 import org.netbeans.lib.cvsclient.file.AbstractFileObject;
 import org.netbeans.lib.cvsclient.file.ICvsFileSystem;
 
-import java.util.Iterator;
 import java.util.LinkedList;
 import java.util.List;
 
@@ -27,7 +26,7 @@ public final class DefaultIgnoreFileFilter
 
        // Fields =================================================================
 
-       private final List patterns = new LinkedList();
+       private final List<IStringPattern> patterns = new LinkedList<>();
 
        // Setup ==================================================================
 
@@ -67,8 +66,7 @@ public final class DefaultIgnoreFileFilter
                final String noneCvsFile = abstractFileObject.getName();
                // current implementation ignores the directory parameter.
                // in future or different implementations can add the directory dependant .cvsignore lists
-               for (Iterator it = patterns.iterator(); it.hasNext();) {
-                       final IStringPattern pattern = (IStringPattern)it.next();
+               for (IStringPattern pattern : patterns) {
                        if (pattern.doesMatch(noneCvsFile)) {
                                return true;
                        }
index 327c6ccd9310701c10a1f69dd87720aa445a1798..0496a5b9ad9ba8e8021c69e70d18a2b11f71703a 100644 (file)
@@ -5,7 +5,6 @@ import org.codehaus.groovy.control.MultipleCompilationErrorsException;
 import org.codehaus.groovy.control.messages.ExceptionMessage;
 
 import java.net.URL;
-import java.util.Iterator;
 import java.util.List;
 
 /**
@@ -24,8 +23,7 @@ public class GrapeRunner {
     }
     catch (MultipleCompilationErrorsException e) {
       List errors = e.getErrorCollector().getErrors();
-      for (Iterator iterator = errors.iterator(); iterator.hasNext();) {
-        Object o = iterator.next();
+      for (Object o : errors) {
         if (o instanceof ExceptionMessage) {
           Exception cause = ((ExceptionMessage)o).getCause();
           String message = cause.getMessage();
@@ -44,8 +42,8 @@ public class GrapeRunner {
     }
 
     URL[] urls = shell.getClassLoader().getURLs();
-    for (int i = 0; i < urls.length; i++) {
-      System.out.println(URL_PREFIX + urls[i]);
+    for (URL url : urls) {
+      System.out.println(URL_PREFIX + url);
     }
   }
 
index 2aba6b9ccf6e210034293a6ec368741b5bdffd77..d03cf655f22f7bb14e17b2b6c3adb5b8bde8c44c 100644 (file)
@@ -200,8 +200,7 @@ public class GriffonSourceInspector {
     List<String> words = new ArrayList<>();
     int i = 0;
     char[] chars = name.toCharArray();
-    for (int j = 0; j < chars.length; j++) {
-      char c = chars[j];
+    for (char c : chars) {
       String w;
       if (i >= words.size()) {
         w = "";
index 8e61c51f3e5757d104d4e76c48163d48a159a1ee..1c1dfa4f8bd577412053d0365bc4cb0aeadd97f1 100644 (file)
@@ -76,8 +76,7 @@ public class GrMemberInfoStorage extends AbstractMemberInfoStorage<GrMember, Psi
   }
 
   private void buildSubClassesMapForList(final PsiClassType[] classesList, GrTypeDefinition aClass) {
-    for (int i = 0; i < classesList.length; i++) {
-      PsiClassType element = classesList[i];
+    for (PsiClassType element : classesList) {
       PsiClass resolved = element.resolve();
       if (resolved instanceof GrTypeDefinition) {
         GrTypeDefinition superClass = (GrTypeDefinition)resolved;
index 04eaae435414220b5c2ea444fa7cec3ba1b274b5..ad7dd82b851d8473786805f4ebbe8cd31c91271d 100644 (file)
@@ -177,8 +177,8 @@ public class HgUpdateTest extends HgCollaborativeTest {
   @Override
   protected HgFile getHgFile(String... filepath) {
     File fileToInclude = projectRepo;
-    for (int i = 0; i < filepath.length; i++) {
-      fileToInclude = new File(fileToInclude, filepath[i]);
+    for (String path : filepath) {
+      fileToInclude = new File(fileToInclude, path);
     }
     return new HgFile(projectRepoVirtualFile, fileToInclude);
   }
index 2c5a2ac84f6607ae177d9570dcee37e277510627..6bdd8dbe64e8fc562952e63f424b562d9bbc17b0 100644 (file)
@@ -566,12 +566,12 @@ public class FunctionExprent extends Exprent {
   }
 
   private static VarType getMaxVarType(VarType[] arr) {
-    int[] types = new int[]{CodeConstants.TYPE_DOUBLE, CodeConstants.TYPE_FLOAT, CodeConstants.TYPE_LONG};
-    VarType[] vartypes = new VarType[]{VarType.VARTYPE_DOUBLE, VarType.VARTYPE_FLOAT, VarType.VARTYPE_LONG};
+    int[] types = {CodeConstants.TYPE_DOUBLE, CodeConstants.TYPE_FLOAT, CodeConstants.TYPE_LONG};
+    VarType[] vartypes = {VarType.VARTYPE_DOUBLE, VarType.VARTYPE_FLOAT, VarType.VARTYPE_LONG};
 
     for (int i = 0; i < types.length; i++) {
-      for (int j = 0; j < arr.length; j++) {
-        if (arr[j].type == types[i]) {
+      for (VarType anArr : arr) {
+        if (anArr.type == types[i]) {
           return vartypes[i];
         }
       }
index 52678fc61880142286bb16b5effcdbe86962075a..62bc3a722bc5e7797c5a9785c543f43f0344ebf7 100644 (file)
@@ -570,9 +570,8 @@ public final class Generator {
     PsiClass beanClass = null;
 
     // find get/set pair and bean class
-    outer: for (int i = 0; i < allGetDataMethods.length; i++) {
-      final PsiMethod _getMethod = allGetDataMethods[i];
-
+    outer:
+    for (final PsiMethod _getMethod : allGetDataMethods) {
       if (!PsiType.VOID.equals(_getMethod.getReturnType())) {
         continue;
       }