You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@felix.apache.org by ri...@apache.org on 2011/01/24 19:08:09 UTC
svn commit: r1062903 -
/felix/trunk/framework/src/main/java/org/apache/felix/framework/Logger.java
Author: rickhall
Date: Mon Jan 24 18:08:09 2011
New Revision: 1062903
URL: http://svn.apache.org/viewvc?rev=1062903&view=rev
Log:
Check for null Bundle in logging code. (FELIX-2800)
Modified:
felix/trunk/framework/src/main/java/org/apache/felix/framework/Logger.java
Modified: felix/trunk/framework/src/main/java/org/apache/felix/framework/Logger.java
URL: http://svn.apache.org/viewvc/felix/trunk/framework/src/main/java/org/apache/felix/framework/Logger.java?rev=1062903&r1=1062902&r2=1062903&view=diff
==============================================================================
--- felix/trunk/framework/src/main/java/org/apache/felix/framework/Logger.java (original)
+++ felix/trunk/framework/src/main/java/org/apache/felix/framework/Logger.java Mon Jan 24 18:08:09 2011
@@ -118,10 +118,20 @@ public class Logger implements ServiceLi
Bundle bundle, ServiceReference sr, int level,
String msg, Throwable throwable)
{
- String s = (sr == null) ? null : "SvcRef " + sr;
- s = (s == null) ? null : s + " Bundle '" + bundle.getBundleId() + "'";
- s = (s == null) ? msg : s + " " + msg;
- s = (throwable == null) ? s : s + " (" + throwable + ")";
+ String s = "";
+ if (sr != null)
+ {
+ s = s + "SvcRef " + sr + " ";
+ }
+ else if (bundle != null)
+ {
+ s = s + "Bundle " + bundle.toString() + " ";
+ }
+ s = s + msg;
+ if (throwable != null)
+ {
+ s = s + " (" + throwable + ")";
+ }
switch (level)
{
case LOG_DEBUG:
@@ -311,4 +321,4 @@ public class Logger implements ServiceLi
}
}
}
-}
\ No newline at end of file
+}