Move commit and tag formatting to CommitBuilder, TagBuilder

These objects should be responsible for their own formatting,
rather than delegating it to some obtuse type called ObjectInserter.

While we are at it, simplify the way we insert these into a database.
Passing in the type and calling format in application code turned
out to be a huge mistake in terms of ease-of-use of the insert API.

Change-Id: Id5bb95ee56aa2a002243e9b7853b84ec8df1d7bf
Signed-off-by: Shawn O. Pearce <spearce@spearce.org>

1 files changed, 3 insertions, 4 deletions

diff --git a/org.eclipse.jgit.junit/src/org/eclipse/jgit/junit/TestRepository.java b/org.eclipse.jgit.junit/src/org/eclipse/jgit/junit/TestRepository.java
index 6e1f1db397..7a95ccd368 100644
--- a/org.eclipse.jgit.junit/src/org/eclipse/jgit/junit/TestRepository.java
+++ b/org.eclipse.jgit.junit/src/org/eclipse/jgit/junit/TestRepository.java
@@ -368,7 +368,7 @@ public class TestRepository<R extends Repository> {
 		c.setMessage("");
 		ObjectId id;
 		try {
-			id = inserter.insert(Constants.OBJ_COMMIT, inserter.format(c));
+			id = inserter.insert(c);
 			inserter.flush();
 		} finally {
 			inserter.release();
@@ -405,7 +405,7 @@ public class TestRepository<R extends Repository> {
 		t.setMessage("");
 		ObjectId id;
 		try {
-			id = inserter.insert(Constants.OBJ_TAG, inserter.format(t));
+			id = inserter.insert(t);
 			inserter.flush();
 		} finally {
 			inserter.release();
@@ -822,8 +822,7 @@ public class TestRepository<R extends Repository> {
 				ObjectId commitId;
 				try {
 					c.setTreeId(tree.writeTree(inserter));
-					commitId = inserter.insert(Constants.OBJ_COMMIT, inserter
-							.format(c));
+					commitId = inserter.insert(c);
 					inserter.flush();
 				} finally {
 					inserter.release();