[Yanel-dev] Re: [Yanel-commits] rev 43260 - public/yanel/trunk/src/impl/java/org/wyona/yanel/impl/resources

[Guillaume Déflache]

michi at wyona.com schrieb:
> Author: michi
> Date: 2009-06-19 10:53:36 +0200 (Fri, 19 Jun 2009)
> New Revision: 43260
> 
> Modified:
>    public/yanel/trunk/src/impl/java/org/wyona/yanel/impl/resources/BasicXMLResource.java
> Log:
> log level improved
> 
> Modified: public/yanel/trunk/src/impl/java/org/wyona/yanel/impl/resources/BasicXMLResource.java
> ===================================================================
> --- public/yanel/trunk/src/impl/java/org/wyona/yanel/impl/resources/BasicXMLResource.java	2009-06-19 08:07:38 UTC (rev 43259)
> +++ public/yanel/trunk/src/impl/java/org/wyona/yanel/impl/resources/BasicXMLResource.java	2009-06-19 08:53:36 UTC (rev 43260)
> @@ -254,13 +254,13 @@
>              for (int i = 0; i < xsltPaths.length; i++) {
>                  // TODO: Use resolver
>                  if (xsltPaths[i].startsWith("file:")) {
> -                    log.warn("Scheme: file (" + xsltPaths[i] + ")");
> +                    log.info("Scheme: file (" + xsltPaths[i] + ")");
>                      xsltHandlers[i] = tf.newTransformerHandler(new StreamSource(new java.io.FileInputStream(xsltPaths[i].substring(5))));
>                  } else if(xsltPaths[i].startsWith("rthtdocs:")) {
> -                    log.warn("Scheme: rthtdocs (" + xsltPaths[i] + ")");
> +                    log.info("Scheme: rthtdocs (" + xsltPaths[i] + ")");
>                      xsltHandlers[i] = tf.newTransformerHandler(new org.wyona.yanel.core.source.RTHtdocsResolver(this).resolve(xsltPaths[i], null));
>                  } else if(xsltPaths[i].startsWith("yanelresource:")) {
> -                    log.warn("Scheme: yanelresource (" + xsltPaths[i] + ")");
> +                    log.info("Scheme: yanelresource (" + xsltPaths[i] + ")");
>                      xsltHandlers[i] = tf.newTransformerHandler(new org.wyona.yanel.core.source.ResourceResolver(this).resolve(xsltPaths[i], null));
>                  } else {
>                      if (xsltPaths[i].indexOf(":/") > 0) {
> 

if (log.isInfoEnabled()) log.info([...]); (and likewise for DEBUG and 
TRACE levels at least) would have been even better, please think about 
it when you come around changing these. Thanks.