From f7cdf7aa07dbd8531db24ab6fe1adc82c701bd9e Mon Sep 17 00:00:00 2001
From: Sebastian Sdorra <sebastian.sdorra@cloudogu.com>
Date: Sat, 16 Jun 2012 12:29:25 +0200
Subject: [PATCH] added logging to diff command builder

---
 .../scm/repository/api/DiffCommandBuilder.java  | 17 +++++++++++++++++
 1 file changed, 17 insertions(+)

diff --git a/scm-core/src/main/java/sonia/scm/repository/api/DiffCommandBuilder.java b/scm-core/src/main/java/sonia/scm/repository/api/DiffCommandBuilder.java
index c277ecb124..35ef6b0359 100644
--- a/scm-core/src/main/java/sonia/scm/repository/api/DiffCommandBuilder.java
+++ b/scm-core/src/main/java/sonia/scm/repository/api/DiffCommandBuilder.java
@@ -38,6 +38,9 @@ package sonia.scm.repository.api;
 import com.google.common.base.Preconditions;
 import com.google.common.io.Closeables;
 
+import org.slf4j.Logger;
+import org.slf4j.LoggerFactory;
+
 import sonia.scm.repository.RepositoryException;
 import sonia.scm.repository.spi.DiffCommand;
 import sonia.scm.repository.spi.DiffCommandRequest;
@@ -60,6 +63,14 @@ import java.io.OutputStream;
 public final class DiffCommandBuilder
 {
 
+  /**
+   * the logger for DiffCommandBuilder
+   */
+  private static final Logger logger =
+    LoggerFactory.getLogger(DiffCommandBuilder.class);
+
+  //~--- constructors ---------------------------------------------------------
+
   /**
    * Constructs a new {@link DiffCommandBuilder}, this constructor should
    * only be called from the {@link RepositoryService}.
@@ -173,6 +184,12 @@ public final class DiffCommandBuilder
     Preconditions.checkNotNull(outputStream, "OutputStream is required");
     Preconditions.checkArgument(request.isValid(),
                                 "path and/or revision is required");
+
+    if (logger.isDebugEnabled())
+    {
+      logger.debug("create diff for {}", request);
+    }
+
     diffCommand.getDiffResult(request, outputStream);
   }