Skip to main content
aboutsummaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorchammer2019-11-12 20:17:19 +0000
committerKarsten Thoms2019-11-14 20:09:57 +0000
commit2d8961f9216a5f2d26b44ae19f75c652ba5b0f48 (patch)
treec56ee445efb5f7bcfd37f7b64cfcd291df2ac165 /org.eclipse.debug.ui/ui/org/eclipse/debug/internal
parentcd94877fcf27071a7aa5bc739e24c00ee2d835f8 (diff)
downloadeclipse.platform.debug-2d8961f9216a5f2d26b44ae19f75c652ba5b0f48.tar.gz
eclipse.platform.debug-2d8961f9216a5f2d26b44ae19f75c652ba5b0f48.tar.xz
eclipse.platform.debug-2d8961f9216a5f2d26b44ae19f75c652ba5b0f48.zip
Use jdk 5 for-each loop
Replace simple uses of Iterator with a corresponding for-loop. Also add missing braces on loops as necessary. Change-Id: I64fa6afd85181b24a5cc3bd13a61aca68dc59a00 Signed-off-by: chammer <carsten.hammer@t-online.de>
Diffstat (limited to 'org.eclipse.debug.ui/ui/org/eclipse/debug/internal')
-rw-r--r--org.eclipse.debug.ui/ui/org/eclipse/debug/internal/ui/model/elements/BreakpointContainerLabelProvider.java12
-rw-r--r--org.eclipse.debug.ui/ui/org/eclipse/debug/internal/ui/viewers/AsynchronousModel.java23
-rw-r--r--org.eclipse.debug.ui/ui/org/eclipse/debug/internal/ui/viewers/AsynchronousTableModel.java32
-rw-r--r--org.eclipse.debug.ui/ui/org/eclipse/debug/internal/ui/viewers/AsynchronousTableViewer.java4
-rw-r--r--org.eclipse.debug.ui/ui/org/eclipse/debug/internal/ui/viewers/AsynchronousViewer.java17
-rw-r--r--org.eclipse.debug.ui/ui/org/eclipse/debug/internal/ui/viewers/breadcrumb/BreadcrumbViewer.java26
-rw-r--r--org.eclipse.debug.ui/ui/org/eclipse/debug/internal/ui/viewers/update/BreakpointContainerProxy.java5
-rw-r--r--org.eclipse.debug.ui/ui/org/eclipse/debug/internal/ui/views/breakpoints/BreakpointContainer.java15
-rw-r--r--org.eclipse.debug.ui/ui/org/eclipse/debug/internal/ui/views/memory/renderings/AsyncTableRenderingCellModifier.java4
9 files changed, 59 insertions, 79 deletions
diff --git a/org.eclipse.debug.ui/ui/org/eclipse/debug/internal/ui/model/elements/BreakpointContainerLabelProvider.java b/org.eclipse.debug.ui/ui/org/eclipse/debug/internal/ui/model/elements/BreakpointContainerLabelProvider.java
index 13f305fe1..d19b878a5 100644
--- a/org.eclipse.debug.ui/ui/org/eclipse/debug/internal/ui/model/elements/BreakpointContainerLabelProvider.java
+++ b/org.eclipse.debug.ui/ui/org/eclipse/debug/internal/ui/model/elements/BreakpointContainerLabelProvider.java
@@ -52,9 +52,8 @@ public class BreakpointContainerLabelProvider extends DebugElementLabelProvider
Object lastSegment = path.getLastSegment();
if (lastSegment instanceof IBreakpointContainer) {
IBreakpointContainer container = (IBreakpointContainer) lastSegment;
- IBreakpoint[] breakpoints = container.getBreakpoints();
- for (int i = 0; i < breakpoints.length; ++i) {
- if (breakpoints[i].isEnabled()) {
+ for (IBreakpoint breakpoint : container.getBreakpoints()) {
+ if (breakpoint.isEnabled()) {
return true;
}
}
@@ -70,15 +69,14 @@ public class BreakpointContainerLabelProvider extends DebugElementLabelProvider
Object lastSegment = path.getLastSegment();
if (lastSegment instanceof IBreakpointContainer) {
IBreakpointContainer container = (IBreakpointContainer) lastSegment;
- IBreakpoint[] breakpoints = container.getBreakpoints();
-
+
// Return true, gray if some breakpoints are enabled and some are disabled.
// return false if all breakpoints are either disabled or all are enabled.
boolean hasEnabled = false;
boolean hasDisabled = false;
- for (int i = 0; i < breakpoints.length; ++i) {
- if (breakpoints[i].isEnabled()) {
+ for (IBreakpoint breakpoint : container.getBreakpoints()) {
+ if (breakpoint.isEnabled()) {
hasEnabled = true;
} else {
hasDisabled = true;
diff --git a/org.eclipse.debug.ui/ui/org/eclipse/debug/internal/ui/viewers/AsynchronousModel.java b/org.eclipse.debug.ui/ui/org/eclipse/debug/internal/ui/viewers/AsynchronousModel.java
index 2c772e184..fdb5cc03c 100644
--- a/org.eclipse.debug.ui/ui/org/eclipse/debug/internal/ui/viewers/AsynchronousModel.java
+++ b/org.eclipse.debug.ui/ui/org/eclipse/debug/internal/ui/viewers/AsynchronousModel.java
@@ -269,8 +269,8 @@ public abstract class AsynchronousModel {
if (nodes == null) {
fElementToNodes.put(element, new ModelNode[] { node});
} else {
- for (int i = 0; i < nodes.length; i++) {
- if (nodes[i] == node) {
+ for (ModelNode mnode : nodes) {
+ if (mnode == node) {
return;
}
}
@@ -388,16 +388,16 @@ public abstract class AsynchronousModel {
ViewerFilter[] filters = getViewer().getFilters();
if (filters != null) {
ArrayList<Object> filtered = new ArrayList<>(elements.length);
- for (int i = 0; i < elements.length; i++) {
+ for (Object element : elements) {
boolean add = true;
- for (int j = 0; j < filters.length; j++) {
- add = filters[j].select(getViewer(), parent, elements[i]);
+ for (ViewerFilter filter : filters) {
+ add = filter.select(getViewer(), parent, element);
if (!add) {
break;
}
}
if (add) {
- filtered.add(elements[i]);
+ filtered.add(element);
}
}
return filtered.toArray();
@@ -585,8 +585,7 @@ public abstract class AsynchronousModel {
mapElement(child, newChildren[i]);
}
}
- for (int i = 0; i < prevKids.length; i++) {
- ModelNode kid = prevKids[i];
+ for (ModelNode kid : prevKids) {
if (kid != null) {
kid.dispose();
unmapNode(kid);
@@ -604,8 +603,8 @@ public abstract class AsynchronousModel {
final ModelNode[] finalUnmap = unmap;
preservingSelection(() -> {
if (finalUnmap != null) {
- for (int i = 0; i < finalUnmap.length; i++) {
- viewer.unmapNode(finalUnmap[i]);
+ for (ModelNode element : finalUnmap) {
+ viewer.unmapNode(element);
}
}
viewer.nodeChildrenChanged(parentNode);
@@ -633,8 +632,8 @@ public abstract class AsynchronousModel {
buf.append('\n');
ModelNode[] childrenNodes = node.getChildrenNodes();
if (childrenNodes != null) {
- for (int i = 0; i < childrenNodes.length; i++) {
- append(buf, childrenNodes[i], level + 1);
+ for (ModelNode childNode : childrenNodes) {
+ append(buf, childNode, level + 1);
}
}
}
diff --git a/org.eclipse.debug.ui/ui/org/eclipse/debug/internal/ui/viewers/AsynchronousTableModel.java b/org.eclipse.debug.ui/ui/org/eclipse/debug/internal/ui/viewers/AsynchronousTableModel.java
index 92247cf22..412b079a4 100644
--- a/org.eclipse.debug.ui/ui/org/eclipse/debug/internal/ui/viewers/AsynchronousTableModel.java
+++ b/org.eclipse.debug.ui/ui/org/eclipse/debug/internal/ui/viewers/AsynchronousTableModel.java
@@ -60,13 +60,13 @@ public class AsynchronousTableModel extends AsynchronousModel {
kids = new ArrayList<>(elements.length);
} else {
kids = new ArrayList<>(elements.length + childrenNodes.length);
- for (int i = 0; i < childrenNodes.length; i++) {
- kids.add(childrenNodes[i].getElement());
+ for (ModelNode childNode : childrenNodes) {
+ kids.add(childNode.getElement());
}
}
- for (int i = 0; i < elements.length; i++) {
- if (!kids.contains(elements[i])) {
- kids.add(elements[i]);
+ for (Object element : elements) {
+ if (!kids.contains(element)) {
+ kids.add(element);
changed = true;
}
}
@@ -103,13 +103,13 @@ public class AsynchronousTableModel extends AsynchronousModel {
kids = new ArrayList<>(elements.length);
} else {
kids = new ArrayList<>(elements.length + childrenNodes.length);
- for (int i = 0; i < childrenNodes.length; i++) {
- kids.add(childrenNodes[i].getElement());
+ for (ModelNode childNode : childrenNodes) {
+ kids.add(childNode.getElement());
}
}
- for (int i = 0; i < elements.length; i++) {
- if (!kids.contains(elements[i])) {
- kids.add(index, elements[i]);
+ for (Object element : elements) {
+ if (!kids.contains(element)) {
+ kids.add(index, element);
index++;
changed = true;
}
@@ -143,11 +143,11 @@ public class AsynchronousTableModel extends AsynchronousModel {
ModelNode[] childrenNodes = getRootNode().getChildrenNodes();
if (childrenNodes != null) {
kids = new ArrayList<>(childrenNodes.length);
- for (int i = 0; i < childrenNodes.length; i++) {
- kids.add(childrenNodes[i].getElement());
+ for (ModelNode childrenNode : childrenNodes) {
+ kids.add(childrenNode.getElement());
}
- for (int i = 0; i < elements.length; i++) {
- if (kids.remove(elements[i])) {
+ for (Object element : elements) {
+ if (kids.remove(element)) {
changed = true;
}
}
@@ -182,9 +182,7 @@ public class AsynchronousTableModel extends AsynchronousModel {
}
List<ModelNode> list = new ArrayList<>();
synchronized (this) {
- ModelNode[] nodes = getNodes(element);
- for (int i = 0; i < nodes.length; i++) {
- ModelNode node = nodes[i];
+ for (ModelNode node : getNodes(element)) {
node.remap(replacement);
list.add(node);
}
diff --git a/org.eclipse.debug.ui/ui/org/eclipse/debug/internal/ui/viewers/AsynchronousTableViewer.java b/org.eclipse.debug.ui/ui/org/eclipse/debug/internal/ui/viewers/AsynchronousTableViewer.java
index a4715aa7f..81c439a02 100644
--- a/org.eclipse.debug.ui/ui/org/eclipse/debug/internal/ui/viewers/AsynchronousTableViewer.java
+++ b/org.eclipse.debug.ui/ui/org/eclipse/debug/internal/ui/viewers/AsynchronousTableViewer.java
@@ -156,8 +156,8 @@ public class AsynchronousTableViewer extends AsynchronousViewer implements Liste
protected List getSelectionFromWidget() {
TableItem[] selection = fTable.getSelection();
List<Object> datas = new ArrayList<>(selection.length);
- for (int i = 0; i < selection.length; i++) {
- datas.add(selection[i].getData());
+ for (TableItem element : selection) {
+ datas.add(element.getData());
}
return datas;
}
diff --git a/org.eclipse.debug.ui/ui/org/eclipse/debug/internal/ui/viewers/AsynchronousViewer.java b/org.eclipse.debug.ui/ui/org/eclipse/debug/internal/ui/viewers/AsynchronousViewer.java
index 54511edf7..aba77c7ab 100644
--- a/org.eclipse.debug.ui/ui/org/eclipse/debug/internal/ui/viewers/AsynchronousViewer.java
+++ b/org.eclipse.debug.ui/ui/org/eclipse/debug/internal/ui/viewers/AsynchronousViewer.java
@@ -190,8 +190,8 @@ public abstract class AsynchronousViewer extends StructuredViewer implements Lis
public void update(Object element) {
ModelNode[] nodes = getModel().getNodes(element);
if (nodes != null) {
- for (int i = 0; i < nodes.length; i++) {
- updateLabel(nodes[i]);
+ for (ModelNode node : nodes) {
+ updateLabel(node);
}
}
}
@@ -421,8 +421,7 @@ public abstract class AsynchronousViewer extends StructuredViewer implements Lis
if (model != null) {
ModelNode[] nodes = model.getNodes(element);
if (nodes != null) {
- for (int i = 0; i < nodes.length; i++) {
- ModelNode node = nodes[i];
+ for (ModelNode node : nodes) {
// get the widget for the node
Widget item = findItem(node);
if (item != null) {
@@ -969,8 +968,8 @@ public abstract class AsynchronousViewer extends StructuredViewer implements Lis
unmapElement(node);
ModelNode[] childrenNodes = node.getChildrenNodes();
if (childrenNodes != null) {
- for (int i = 0; i < childrenNodes.length; i++) {
- unmapNode(childrenNodes[i]);
+ for (ModelNode childNode : childrenNodes) {
+ unmapNode(childNode);
}
}
}
@@ -983,8 +982,7 @@ public abstract class AsynchronousViewer extends StructuredViewer implements Lis
protected ModelNode findNode(Widget widget) {
ModelNode[] nodes = getModel().getNodes(widget.getData());
if (nodes != null) {
- for (int i = 0; i < nodes.length; i++) {
- ModelNode node = nodes[i];
+ for (ModelNode node : nodes) {
Widget item = findItem(node);
if (widget == item) {
return node;
@@ -1031,8 +1029,7 @@ public abstract class AsynchronousViewer extends StructuredViewer implements Lis
if (parentItem != null) {
ModelNode[] nodes = getModel().getNodes(parentItem.getData());
if (nodes != null) {
- for (int i = 0; i < nodes.length; i++) {
- ModelNode parentNode = nodes[i];
+ for (ModelNode parentNode : nodes) {
Widget parentWidget = findItem(parentNode);
if (parentWidget == parentItem) {
ModelNode[] childrenNodes = parentNode.getChildrenNodes();
diff --git a/org.eclipse.debug.ui/ui/org/eclipse/debug/internal/ui/viewers/breadcrumb/BreadcrumbViewer.java b/org.eclipse.debug.ui/ui/org/eclipse/debug/internal/ui/viewers/breadcrumb/BreadcrumbViewer.java
index 64d38a7f4..f3b1fd4a3 100644
--- a/org.eclipse.debug.ui/ui/org/eclipse/debug/internal/ui/viewers/breadcrumb/BreadcrumbViewer.java
+++ b/org.eclipse.debug.ui/ui/org/eclipse/debug/internal/ui/viewers/breadcrumb/BreadcrumbViewer.java
@@ -170,8 +170,7 @@ public abstract class BreadcrumbViewer extends StructuredViewer {
* @return true if any of the items in the viewer is expanded
*/
public boolean isDropDownOpen() {
- for (int i= 0, size= fBreadcrumbItems.size(); i < size; i++) {
- BreadcrumbItem item= fBreadcrumbItems.get(i);
+ for (BreadcrumbItem item : fBreadcrumbItems) {
if (item.isMenuShown()) {
return true;
}
@@ -187,8 +186,7 @@ public abstract class BreadcrumbViewer extends StructuredViewer {
* @return the drop downs shell or <code>null</code>
*/
public Shell getDropDownShell() {
- for (int i= 0, size= fBreadcrumbItems.size(); i < size; i++) {
- BreadcrumbItem item= fBreadcrumbItems.get(i);
+ for (BreadcrumbItem item : fBreadcrumbItems) {
if (item.isMenuShown()) {
return item.getDropDownShell();
}
@@ -268,8 +266,7 @@ public abstract class BreadcrumbViewer extends StructuredViewer {
return null;
}
- for (int i= 0, size= fBreadcrumbItems.size(); i < size; i++) {
- BreadcrumbItem item= fBreadcrumbItems.get(i);
+ for (BreadcrumbItem item : fBreadcrumbItems) {
if (item.getData() == element || element.equals(item.getData())) {
return item;
}
@@ -329,9 +326,9 @@ public abstract class BreadcrumbViewer extends StructuredViewer {
protected TreePath getTreePathFromItem(BreadcrumbItem item) {
List<Object> elements = new ArrayList<>(fBreadcrumbItems.size());
- for (int i = 0; i < fBreadcrumbItems.size(); i++) {
- elements.add( fBreadcrumbItems.get(i).getData() );
- if (fBreadcrumbItems.get(i).equals(item)) {
+ for (BreadcrumbItem currItem : fBreadcrumbItems) {
+ elements.add( currItem.getData() );
+ if (currItem.equals(item)) {
return new TreePath(elements.toArray());
}
}
@@ -368,9 +365,8 @@ public abstract class BreadcrumbViewer extends StructuredViewer {
BreadcrumbItem item= (BreadcrumbItem) doFindItem(element);
if (item == null || element != null && element.equals(getInput())) {
- for (int i= 0, size= fBreadcrumbItems.size(); i < size; i++) {
- BreadcrumbItem item1= fBreadcrumbItems.get(i);
- layoutChanged = refreshItem(item1) || layoutChanged;
+ for (BreadcrumbItem currItem : fBreadcrumbItems) {
+ layoutChanged = refreshItem(currItem) || layoutChanged;
}
} else {
layoutChanged = refreshItem(item) || layoutChanged;
@@ -393,8 +389,7 @@ public abstract class BreadcrumbViewer extends StructuredViewer {
BreadcrumbItem focusItem= null;
// Unselect the currently selected items, and remember the focused item.
- for (int i= 0, size= fBreadcrumbItems.size(); i < size; i++) {
- BreadcrumbItem item= fBreadcrumbItems.get(i);
+ for (BreadcrumbItem item : fBreadcrumbItems) {
if (item.hasFocus()) {
focusItem= item;
}
@@ -729,8 +724,7 @@ public abstract class BreadcrumbViewer extends StructuredViewer {
*/
private int getCurrentWidth() {
int result= 0;
- for (int i= 0, size= fBreadcrumbItems.size(); i < size; i++) {
- BreadcrumbItem viewer= fBreadcrumbItems.get(i);
+ for (BreadcrumbItem viewer : fBreadcrumbItems) {
result+= viewer.getWidth();
}
diff --git a/org.eclipse.debug.ui/ui/org/eclipse/debug/internal/ui/viewers/update/BreakpointContainerProxy.java b/org.eclipse.debug.ui/ui/org/eclipse/debug/internal/ui/viewers/update/BreakpointContainerProxy.java
index ec9e20a17..c54539e26 100644
--- a/org.eclipse.debug.ui/ui/org/eclipse/debug/internal/ui/viewers/update/BreakpointContainerProxy.java
+++ b/org.eclipse.debug.ui/ui/org/eclipse/debug/internal/ui/viewers/update/BreakpointContainerProxy.java
@@ -45,10 +45,9 @@ public class BreakpointContainerProxy extends AbstractModelProxy implements IChe
@Override
public boolean setChecked(IPresentationContext context, Object viewerInput, TreePath path, boolean checked) {
boolean atLeastOne = false;
- IBreakpoint[] breakpoints = fContainer.getBreakpoints();
- for (int i = 0; i < breakpoints.length; ++i) {
+ for (IBreakpoint breakpoint : fContainer.getBreakpoints()) {
try {
- breakpoints[i].setEnabled(checked);
+ breakpoint.setEnabled(checked);
atLeastOne = true;
} catch (CoreException e) {}
}
diff --git a/org.eclipse.debug.ui/ui/org/eclipse/debug/internal/ui/views/breakpoints/BreakpointContainer.java b/org.eclipse.debug.ui/ui/org/eclipse/debug/internal/ui/views/breakpoints/BreakpointContainer.java
index 3488c24c9..137a28b14 100644
--- a/org.eclipse.debug.ui/ui/org/eclipse/debug/internal/ui/views/breakpoints/BreakpointContainer.java
+++ b/org.eclipse.debug.ui/ui/org/eclipse/debug/internal/ui/views/breakpoints/BreakpointContainer.java
@@ -131,13 +131,12 @@ public class BreakpointContainer extends ElementContentProvider implements IAdap
if (fNesting != null && fNesting.length > 0) {
IAdaptable[] emptyCategories = fNesting[0].getCategories();
if (emptyCategories != null) {
- for (int i = 0; i < emptyCategories.length; i++) {
- IAdaptable empty = emptyCategories[i];
- BreakpointContainer container = findExistingContainer(fChildContainers, empty);
+ for (IAdaptable category : emptyCategories) {
+ BreakpointContainer container = findExistingContainer(fChildContainers, category);
if (container == null) {
IBreakpointOrganizer[] siblings = new IBreakpointOrganizer[fNesting.length - 1];
System.arraycopy(fNesting, 1, siblings, 0, siblings.length);
- container = new BreakpointContainer(this, empty, fNesting[0], fComparator, siblings);
+ container = new BreakpointContainer(this, category, fNesting[0], fComparator, siblings);
insertChildContainer(container);
container.fDefaultContainer = true;
@@ -276,11 +275,8 @@ public class BreakpointContainer extends ElementContentProvider implements IAdap
IBreakpointOrganizer organizer = fNesting[0];
// get the breakpoint categories from the organizer
- IAdaptable[] categories = getCategories(breakpoint, organizer);
-
- for (int i = 0; i < categories.length; ++i) {
+ for (IAdaptable category : getCategories(breakpoint, organizer)) {
ModelDelta childDelta = null;
- IAdaptable category = categories[i];
BreakpointContainer container = findExistingContainer(fChildContainers, category);
// create a new container if it doesn't exist
@@ -595,8 +591,7 @@ public class BreakpointContainer extends ElementContentProvider implements IAdap
return new BreakpointContainer[]{this};
}
ArrayList<BreakpointContainer> list = new ArrayList<>();
- for (int i = 0; i < containers.length; i++) {
- BreakpointContainer container = containers[i];
+ for (BreakpointContainer container : containers) {
BreakpointContainer[] subcontainers = container.getContainers(breakpoint);
if (subcontainers != null) {
Collections.addAll(list, subcontainers);
diff --git a/org.eclipse.debug.ui/ui/org/eclipse/debug/internal/ui/views/memory/renderings/AsyncTableRenderingCellModifier.java b/org.eclipse.debug.ui/ui/org/eclipse/debug/internal/ui/views/memory/renderings/AsyncTableRenderingCellModifier.java
index 9312bf799..801b2c7d8 100644
--- a/org.eclipse.debug.ui/ui/org/eclipse/debug/internal/ui/views/memory/renderings/AsyncTableRenderingCellModifier.java
+++ b/org.eclipse.debug.ui/ui/org/eclipse/debug/internal/ui/views/memory/renderings/AsyncTableRenderingCellModifier.java
@@ -94,8 +94,8 @@ public class AsyncTableRenderingCellModifier implements ICellModifier {
return fCustomModifier.canModify(mElement, null);
}
- for (int i = 0; i < bytes.length; i++) {
- if (!bytes[i].isWritable()) {
+ for (MemoryByte b : bytes) {
+ if (!b.isWritable()) {
canModify = false;
}
}

Back to the top