From cbe6840efecf87e53a687cdce6fbdf84c6ab8a46 Mon Sep 17 00:00:00 2001
From: James Moger <james.moger@gitblit.com>
Date: Thu, 19 Sep 2013 08:40:59 -0400
Subject: [PATCH] Documentation

---
 src/main/java/com/gitblit/utils/DiffUtils.java |   14 ++++++--------
 1 files changed, 6 insertions(+), 8 deletions(-)

diff --git a/src/main/java/com/gitblit/utils/DiffUtils.java b/src/main/java/com/gitblit/utils/DiffUtils.java
index 04b5b0b..3844d10 100644
--- a/src/main/java/com/gitblit/utils/DiffUtils.java
+++ b/src/main/java/com/gitblit/utils/DiffUtils.java
@@ -16,6 +16,7 @@
 package com.gitblit.utils;
 
 import java.io.ByteArrayOutputStream;
+import java.text.MessageFormat;
 import java.util.ArrayList;
 import java.util.List;
 
@@ -52,7 +53,7 @@
 	 * Enumeration for the diff output types.
 	 */
 	public static enum DiffOutputType {
-		PLAIN, GITWEB, GITBLIT;
+		PLAIN, HTML;
 
 		public static DiffOutputType forName(String name) {
 			for (DiffOutputType type : values()) {
@@ -129,10 +130,7 @@
 			RawTextComparator cmp = RawTextComparator.DEFAULT;
 			DiffFormatter df;
 			switch (outputType) {
-			case GITWEB:
-				df = new GitWebDiffFormatter(os);
-				break;
-			case GITBLIT:
+			case HTML:
 				df = new GitBlitDiffFormatter(os);
 				break;
 			case PLAIN:
@@ -171,9 +169,9 @@
 			} else {
 				df.format(diffEntries);
 			}
-			if (df instanceof GitWebDiffFormatter) {
+			if (df instanceof GitBlitDiffFormatter) {
 				// workaround for complex private methods in DiffFormatter
-				diff = ((GitWebDiffFormatter) df).getHtml();
+				diff = ((GitBlitDiffFormatter) df).getHtml();
 			} else {
 				diff = os.toString();
 			}
@@ -274,7 +272,7 @@
 				lines.add(line);
 			}
 		} catch (Throwable t) {
-			LOGGER.error("failed to generate blame!", t);
+			LOGGER.error(MessageFormat.format("failed to generate blame for {0} {1}!", blobPath, objectId), t);
 		}
 		return lines;
 	}

--
Gitblit v1.9.1