Fixed crash with GUI when toggling the Grid size quickly, fixes #1440

CHANGELOG.md

@@ -6,6 +6,7 @@
 - Fixed network not disconnecting when Controller is broken (raoulvdberge)
 - Fixed bug where when multiple Fortune Upgrades are inserted, it chooses the first Fortune Upgrade instead of the highest one (raoulvdberge)
 - Fixed some translations having too big "Craft" text (raoulvdberge)
+- Fixed crash with GUI when toggling the Grid size quickly (raoulvdberge)
 - The Fortune Upgrade doesn't use NBT anymore to store the fortune level (raoulvdberge)
 - Pressing SHIFT over an item in the Grid will no longer display the full unformatted count, instead, use CTRL + SHIFT and it will be displayed in the tooltip (raoulvdberge)
 

src/main/java/com/raoulvdberge/refinedstorage/gui/GuiBase.java

@@ -162,7 +162,9 @@ public abstract class GuiBase extends GuiContainer {
 
         String sideButtonTooltip = null;
 
-        for (GuiButton button : buttonList) {
+        for (int i = 0; i < buttonList.size(); ++i) {
+            GuiButton button = buttonList.get(i);
+
             if (button instanceof SideButton && ((SideButton) button).isHovered()) {
                 sideButtonTooltip = ((SideButton) button).getTooltip();
             }
@@ -327,10 +329,6 @@ public abstract class GuiBase extends GuiContainer {
         GlStateManager.enableLighting();
     }
 
-    public void drawTooltip(int x, int y, List<String> lines) {
-        drawTooltip(ItemStack.EMPTY, x, y, lines);
-    }
-
     public void drawTexture(int x, int y, int textureX, int textureY, int width, int height) {
         drawTexturedModalRect(x, y, textureX, textureY, width, height);
     }