logX usage

weld · Mar 15, 2010 · d100fba · d100fba
1 parent fd43226
commit d100fba
Showing 1 changed file with 9 additions and 2 deletions.
diff --git a/impl/src/main/java/org/jboss/weld/util/serviceProvider/DefaultServiceLoader.java b/impl/src/main/java/org/jboss/weld/util/serviceProvider/DefaultServiceLoader.java
@@ -40,6 +40,8 @@
 import org.jboss.weld.util.collections.EnumerationList;
 import org.jboss.weld.util.reflection.SecureReflections;
 import org.slf4j.cal10n.LocLogger;
+import org.slf4j.ext.XLogger;
+import org.slf4j.ext.XLogger.Level;
 
 /**
  * This class handles looking up service providers on the class path. It
@@ -60,7 +62,8 @@
 public class DefaultServiceLoader<S> implements Iterable<S>
 {
    private static LocLogger log = loggerFactory().getLogger(UTIL);
-
+   private static XLogger logX = loggerFactory().getXLogger(UTIL);
+
    private static final String SERVICES = "META-INF/services";
 
    /**
@@ -204,7 +207,8 @@ private void loadServiceFile(URL serviceFile)
       }
       catch (IOException e)
       {
-         throw new ForbiddenStateException(COULD_NOT_READ_SERVICES_FILE, serviceFile, e);
+         logX.throwing(Level.ERROR, e);
+         throw new ForbiddenStateException(COULD_NOT_READ_SERVICES_FILE, serviceFile);
       }
       finally
       {
@@ -251,10 +255,12 @@ private Class<? extends S> loadClass(String serviceClassName)
       }
       catch (ClassNotFoundException e)
       {
+         logX.throwing(Level.ERROR, e);
          throw new ForbiddenStateException(EXTENSION_CLASS_NOT_FOUND, serviceClassName);
       }
       catch (ClassCastException e)
       {
+         logX.throwing(Level.ERROR, e);
          throw new ForbiddenStateException(DECLARED_EXTENSION_DOES_NOT_IMPLEMENT_EXTENSION, serviceClassName);
       }
       return serviceClass;
@@ -269,6 +275,7 @@ private S prepareInstance(Class<? extends S> serviceClass)
       }
       catch (Exception e)
       {
+         logX.throwing(Level.ERROR, e);
          throw new ForbiddenStateException(SECURITY_EXCEPTION_SCANNING, serviceClass, e);
       }
    }