From 49c33ff2d04c04dd21c9244d673b934ac340eacb Mon Sep 17 00:00:00 2001
From: Artur Signell <artur@vaadin.com>
Date: Tue, 20 Mar 2012 14:44:06 +0200
Subject: [PATCH] Code cleanup

---
 src/com/vaadin/ui/AbstractComponent.java | 18 ++++++++++--------
 1 file changed, 10 insertions(+), 8 deletions(-)

diff --git a/src/com/vaadin/ui/AbstractComponent.java b/src/com/vaadin/ui/AbstractComponent.java
index 4d12e0ed27..ad91f342ba 100644
--- a/src/com/vaadin/ui/AbstractComponent.java
+++ b/src/com/vaadin/ui/AbstractComponent.java
@@ -438,14 +438,16 @@ public abstract class AbstractComponent implements Component, MethodEventSource
      * @see com.vaadin.ui.Component#setVisible(boolean)
      */
     public void setVisible(boolean visible) {
-        if (getState().isVisible() != visible) {
-            getState().setVisible(visible);
-            requestRepaint();
-            if (getParent() != null) {
-                // Must always repaint the parent (at least the hierarchy) when
-                // visibility of a child component changes.
-                getParent().requestRepaint();
-            }
+        if (getState().isVisible() == visible) {
+            return;
+        }
+
+        getState().setVisible(visible);
+        requestRepaint();
+        if (getParent() != null) {
+            // Must always repaint the parent (at least the hierarchy) when
+            // visibility of a child component changes.
+            getParent().requestRepaint();
         }
     }
 
-- 
2.39.5