From fb27eea8ee7254b19f61588927c2fa3ea2225779 Mon Sep 17 00:00:00 2001 From: Dmitry Jemerov Date: Tue, 21 Feb 2012 21:04:20 +0100 Subject: [PATCH] cleanup --- .../com/intellij/ui/tabs/impl/ActionPanel.java | 16 +++++----------- 1 file changed, 5 insertions(+), 11 deletions(-) diff --git a/platform/platform-api/src/com/intellij/ui/tabs/impl/ActionPanel.java b/platform/platform-api/src/com/intellij/ui/tabs/impl/ActionPanel.java index 9dcd06843d5a..838a6a1e5095 100644 --- a/platform/platform-api/src/com/intellij/ui/tabs/impl/ActionPanel.java +++ b/platform/platform-api/src/com/intellij/ui/tabs/impl/ActionPanel.java @@ -1,5 +1,5 @@ /* - * Copyright 2000-2009 JetBrains s.r.o. + * Copyright 2000-2012 JetBrains s.r.o. * * Licensed under the Apache License, Version 2.0 (the "License"); * you may not use this file except in compliance with the License. @@ -27,14 +27,11 @@ import javax.swing.*; import java.awt.*; import java.awt.event.MouseEvent; import java.util.ArrayList; -import java.util.Iterator; import java.util.List; public class ActionPanel extends NonOpaquePanel { - private final ActionGroup myGroup; private final List myButtons = new ArrayList(); - private final TabInfo myTabInfo; private final JBTabsImpl myTabs; private boolean myAutoHide; @@ -44,9 +41,8 @@ public class ActionPanel extends NonOpaquePanel { public ActionPanel(JBTabsImpl tabs, TabInfo tabInfo, Pass pass) { myTabs = tabs; - myTabInfo = tabInfo; - myGroup = tabInfo.getTabLabelActions() != null ? tabInfo.getTabLabelActions() : new DefaultActionGroup(); - AnAction[] children = myGroup.getChildren(null); + ActionGroup group = tabInfo.getTabLabelActions() != null ? tabInfo.getTabLabelActions() : new DefaultActionGroup(); + AnAction[] children = group.getChildren(null); final NonOpaquePanel wrapper = new NonOpaquePanel(new BorderLayout()); wrapper.add(Box.createHorizontalStrut(myGap), BorderLayout.WEST); @@ -96,8 +92,7 @@ public class ActionPanel extends NonOpaquePanel { public void setAutoHide(final boolean autoHide) { myAutoHide = autoHide; - for (Iterator iterator = myButtons.iterator(); iterator.hasNext();) { - ActionButton each = iterator.next(); + for (ActionButton each : myButtons) { each.setAutoHide(myAutoHide); } } @@ -108,8 +103,7 @@ public class ActionPanel extends NonOpaquePanel { } public void toggleShowActions(final boolean show) { - for (Iterator iterator = myButtons.iterator(); iterator.hasNext();) { - ActionButton each = iterator.next(); + for (ActionButton each : myButtons) { each.toggleShowActions(show); } }