Merge of (#10563) to Vaadin 7.

Use template strings and log level checks to improve performance when
logging is disabled.

Change-Id: I48d370f523c6361ce8704afc7a0903412b31b2a0

1 files changed, 5 insertions, 2 deletions

diff --git a/server/src/com/vaadin/ui/TreeTable.java b/server/src/com/vaadin/ui/TreeTable.java
index b5c026c9df..e150db9423 100644
--- a/server/src/com/vaadin/ui/TreeTable.java
+++ b/server/src/com/vaadin/ui/TreeTable.java
@@ -23,6 +23,7 @@ import java.util.Collections;
 import java.util.HashSet;
 import java.util.List;
 import java.util.Map;
+import java.util.logging.Level;
 import java.util.logging.Logger;
 
 import com.vaadin.data.Collapsible;
@@ -252,9 +253,11 @@ public class TreeTable extends Table implements Hierarchical {
             boolean removed = openItems.remove(itemId);
             if (!removed) {
                 openItems.add(itemId);
-                getLogger().finest("Item " + itemId + " is now expanded");
+                getLogger().log(Level.FINEST, "Item {0} is now expanded",
+                        itemId);
             } else {
-                getLogger().finest("Item " + itemId + " is now collapsed");
+                getLogger().log(Level.FINEST, "Item {0} is now collapsed",
+                        itemId);
             }
             clearPreorderCache();
         }