commit 9af5eaa1d0210e910056846ea4970b49648e3ebf
author Guillaume Nodet <gnodet@apache.org> Wed Jul 25 09:52:33 2012 +0000
committer Guillaume Nodet <gnodet@apache.org> Wed Jul 25 09:52:33 2012 +0000
tree 33c533b4d0edf20f9e4def04e208936fbad954e1
parent 19a054e3ba9758e6b45e75f259fd24c1f9f4ed4b

[FELIX-3609] Optimization: avoid unnecessary toString() computations

git-svn-id: https://svn.apache.org/repos/asf/felix/trunk@1365507 13f79535-47bb-0310-9956-ffa450edef68

framework/src/main/java/org/apache/felix/framework/StatefulResolver.java

diff --git a/framework/src/main/java/org/apache/felix/framework/StatefulResolver.java b/framework/src/main/java/org/apache/felix/framework/StatefulResolver.java
index 376ffcb..cc9a387 100644
--- a/framework/src/main/java/org/apache/felix/framework/StatefulResolver.java
+++ b/framework/src/main/java/org/apache/felix/framework/StatefulResolver.java
@@ -761,6 +761,8 @@
     private void markResolvedRevisions(Map<BundleRevision, List<ResolverWire>> wireMap)
         throws ResolveException
     {
+        boolean debugLog = m_felix.getLogger().getLogLevel() >= Logger.LOG_DEBUG;
+
         // DO THIS IN THREE PASSES:
         // 1. Aggregate fragments per host.
         // 2. Attach wires and fragments to hosts.
@@ -841,13 +843,19 @@
 
                     if (Util.isFragment(revision))
                     {
-                        m_felix.getLogger().log(
-                            Logger.LOG_DEBUG,
-                            "FRAGMENT WIRE: " + rw.toString());
+                        if (debugLog)
+                        {
+                            m_felix.getLogger().log(
+                                Logger.LOG_DEBUG,
+                                "FRAGMENT WIRE: " + rw.toString());
+                        }
                     }
                     else
                     {
-                        m_felix.getLogger().log(Logger.LOG_DEBUG, "WIRE: " + rw.toString());
+                        if (debugLog)
+                        {
+                            m_felix.getLogger().log(Logger.LOG_DEBUG, "WIRE: " + rw.toString());
+                        }
 
                         if (rw.getCapability().getNamespace()
                             .equals(BundleRevision.PACKAGE_NAMESPACE))