Author: simonetripodi
Date: Mon Apr  4 18:51:16 2011
New Revision: 1088719

URL: http://svn.apache.org/viewvc?rev=1088719&view=rev
Log:
fixed checkstyle violations

Modified:
    
commons/proper/discovery/trunk/src/java/org/apache/commons/discovery/tools/Service.java

Modified: 
commons/proper/discovery/trunk/src/java/org/apache/commons/discovery/tools/Service.java
URL: 
http://svn.apache.org/viewvc/commons/proper/discovery/trunk/src/java/org/apache/commons/discovery/tools/Service.java?rev=1088719&r1=1088718&r2=1088719&view=diff
==============================================================================
--- 
commons/proper/discovery/trunk/src/java/org/apache/commons/discovery/tools/Service.java
 (original)
+++ 
commons/proper/discovery/trunk/src/java/org/apache/commons/discovery/tools/Service.java
 Mon Apr  4 18:51:16 2011
@@ -60,8 +60,11 @@ public class Service
      * sun/jdk1.3.1/docs/guide/jar/jar.html#Service Provider,
      * Except this uses <code>Enumeration</code>
      * instead of <code>Interator</code>.
-     * 
-     * @return Enumeration of class instances (<code>Object</code>)
+     *
+     * @param <T> Service Provider Interface type
+     * @param <S> Any type extends the SPI type
+     * @param spiClass Service Provider Interface Class
+     * @return Enumeration of class instances ({@code S})
      */
     public static <T, S extends T> Enumeration<S> providers(Class<T> spiClass) 
{
         return providers(new SPInterface<T>(spiClass), null);
@@ -69,10 +72,13 @@ public class Service
 
     /**
      * This version lets you specify constructor arguments..
-     * 
+     *
+     * @param <T> Service Provider Interface type
+     * @param <S> Any type extends the SPI type
      * @param spi SPI to look for and load.
      * @param loaders loaders to use in search.
      *        If <code>null</code> then use ClassLoaders.getAppLoaders().
+     * @return Enumeration of class instances ({@code S})
      */
     public static <T, S extends T> Enumeration<S> providers(final 
SPInterface<T> spi,
                                         ClassLoaders loaders)


Reply via email to