You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@commons.apache.org by si...@apache.org on 2011/04/05 23:51:06 UTC
svn commit: r1089255 - in
/commons/proper/discovery/trunk/src/java/org/apache/commons/discovery:
log/DiscoveryLogFactory.java log/SimpleLog.java tools/DiscoverClass.java
Author: simonetripodi
Date: Tue Apr 5 21:51:05 2011
New Revision: 1089255
URL: http://svn.apache.org/viewvc?rev=1089255&view=rev
Log:
fixed checkstyle violations
Modified:
commons/proper/discovery/trunk/src/java/org/apache/commons/discovery/log/DiscoveryLogFactory.java
commons/proper/discovery/trunk/src/java/org/apache/commons/discovery/log/SimpleLog.java
commons/proper/discovery/trunk/src/java/org/apache/commons/discovery/tools/DiscoverClass.java
Modified: commons/proper/discovery/trunk/src/java/org/apache/commons/discovery/log/DiscoveryLogFactory.java
URL: http://svn.apache.org/viewvc/commons/proper/discovery/trunk/src/java/org/apache/commons/discovery/log/DiscoveryLogFactory.java?rev=1089255&r1=1089254&r2=1089255&view=diff
==============================================================================
--- commons/proper/discovery/trunk/src/java/org/apache/commons/discovery/log/DiscoveryLogFactory.java (original)
+++ commons/proper/discovery/trunk/src/java/org/apache/commons/discovery/log/DiscoveryLogFactory.java Tue Apr 5 21:51:05 2011
@@ -128,8 +128,9 @@ public class DiscoveryLogFactory {
// now, go back and reset loggers for all current classes..
for (Class<?> clazz : classRegistry.values()) {
- if (log.isDebugEnabled())
+ if (log.isDebugEnabled()) {
log.debug("Reset Log for: " + clazz.getName());
+ }
Method setLog = null;
Modified: commons/proper/discovery/trunk/src/java/org/apache/commons/discovery/log/SimpleLog.java
URL: http://svn.apache.org/viewvc/commons/proper/discovery/trunk/src/java/org/apache/commons/discovery/log/SimpleLog.java?rev=1089255&r1=1089254&r2=1089255&view=diff
==============================================================================
--- commons/proper/discovery/trunk/src/java/org/apache/commons/discovery/log/SimpleLog.java (original)
+++ commons/proper/discovery/trunk/src/java/org/apache/commons/discovery/log/SimpleLog.java Tue Apr 5 21:51:05 2011
@@ -241,8 +241,9 @@ public class SimpleLog implements Log {
// print to System.err
out.println(buf.toString());
- if (t != null)
+ if (t != null) {
t.printStackTrace(System.err);
+ }
}
// -------------------------------------------------------- Log Implementation
Modified: commons/proper/discovery/trunk/src/java/org/apache/commons/discovery/tools/DiscoverClass.java
URL: http://svn.apache.org/viewvc/commons/proper/discovery/trunk/src/java/org/apache/commons/discovery/tools/DiscoverClass.java?rev=1089255&r1=1089254&r2=1089255&view=diff
==============================================================================
--- commons/proper/discovery/trunk/src/java/org/apache/commons/discovery/tools/DiscoverClass.java (original)
+++ commons/proper/discovery/trunk/src/java/org/apache/commons/discovery/tools/DiscoverClass.java Tue Apr 5 21:51:05 2011
@@ -472,7 +472,7 @@ public class DiscoverClass {
/**
* Create new instance of class implementing SPI.
*
- * @param <S> Any class extending T
+ * @param <T> The SPI type
* @param loaders The class loaders holder
* @param spi Service Provider Interface Class.
* @param properties Used to determine name of SPI implementation,