From d35baa5a48c03f7a0f00cdfc08c8d0d6ed7e5455 Mon Sep 17 00:00:00 2001 From: Robert von Burg Date: Tue, 24 Dec 2013 02:42:49 +0100 Subject: [PATCH] [Minor] set some loggers to debug --- .../java/ch/eitchnet/xmlpers/api/FileIo.java | 26 ++++++++++++------- .../xmlpers/objref/LockableObject.java | 6 +++-- 2 files changed, 21 insertions(+), 11 deletions(-) diff --git a/src/main/java/ch/eitchnet/xmlpers/api/FileIo.java b/src/main/java/ch/eitchnet/xmlpers/api/FileIo.java index f05aac811..23f917e4f 100644 --- a/src/main/java/ch/eitchnet/xmlpers/api/FileIo.java +++ b/src/main/java/ch/eitchnet/xmlpers/api/FileIo.java @@ -95,8 +95,10 @@ public class FileIo { throw new XmlException(msg, e); } - String msg = "Wrote SAX to {0}"; //$NON-NLS-1$ - logger.info(MessageFormat.format(msg, this.path.getAbsolutePath())); + if (logger.isDebugEnabled()) { + String msg = "Wrote SAX to {0}"; //$NON-NLS-1$ + logger.info(MessageFormat.format(msg, this.path.getAbsolutePath())); + } } public void readSax(PersistenceContext ctx) { @@ -110,8 +112,10 @@ public class FileIo { DefaultHandler defaultHandler = saxParser.getDefaultHandler(); sp.parse(this.path, defaultHandler); - String msg = "SAX parsed file {0}"; //$NON-NLS-1$ - logger.info(MessageFormat.format(msg, this.path.getAbsolutePath())); + if (logger.isDebugEnabled()) { + String msg = "SAX parsed file {0}"; //$NON-NLS-1$ + logger.info(MessageFormat.format(msg, this.path.getAbsolutePath())); + } ctx.setObject(saxParser.getObject()); @@ -157,8 +161,10 @@ public class FileIo { Source xmlSource = new DOMSource(document); transformer.transform(xmlSource, result); - String msg = MessageFormat.format("Wrote DOM to {0}", this.path.getAbsolutePath()); //$NON-NLS-1$ - logger.info(msg); + if (logger.isDebugEnabled()) { + String msg = MessageFormat.format("Wrote DOM to {0}", this.path.getAbsolutePath()); //$NON-NLS-1$ + logger.info(msg); + } } catch (TransformerFactoryConfigurationError | TransformerException e) { @@ -183,9 +189,11 @@ public class FileIo { DomParser domParser = ctx.getParserFactor().getDomParser(); domParser.fromDom(document); - String msg = "DOM parsed file {0}"; //$NON-NLS-1$ - msg = MessageFormat.format(msg, this.path.getAbsolutePath()); - logger.info(msg); + if (logger.isDebugEnabled()) { + String msg = "DOM parsed file {0}"; //$NON-NLS-1$ + msg = MessageFormat.format(msg, this.path.getAbsolutePath()); + logger.info(msg); + } ctx.setObject(domParser.getObject()); diff --git a/src/main/java/ch/eitchnet/xmlpers/objref/LockableObject.java b/src/main/java/ch/eitchnet/xmlpers/objref/LockableObject.java index caa0316ee..760b8234f 100644 --- a/src/main/java/ch/eitchnet/xmlpers/objref/LockableObject.java +++ b/src/main/java/ch/eitchnet/xmlpers/objref/LockableObject.java @@ -51,7 +51,8 @@ public class LockableObject { msg = MessageFormat.format(msg, StringHelper.formatMillisecondsDuration(tryLockTime), this.toString()); throw new XmlPersistenceException(msg); } - logger.info("locked " + this.toString()); //$NON-NLS-1$ + if (logger.isDebugEnabled()) + logger.debug("locked " + this.toString()); //$NON-NLS-1$ } catch (InterruptedException e) { throw new XmlPersistenceException("Thread interrupted: " + e.getMessage(), e); //$NON-NLS-1$ } @@ -62,6 +63,7 @@ public class LockableObject { */ public void unlock() { this.lock.unlock(); - logger.info("unlocking " + this.toString()); //$NON-NLS-1$ + if (logger.isDebugEnabled()) + logger.debug("unlocking " + this.toString()); //$NON-NLS-1$ } }