Remove getRawSize, getRawType from ObjectLoader

These were only used by PackWriter to help it filter object
representations.  Their only user disappeared when we rewrote the
object selection code path to use the new representation type.

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

8 files changed, 1 insertions, 93 deletions

diff --git a/org.eclipse.jgit/src/org/eclipse/jgit/lib/ObjectLoader.java b/org.eclipse.jgit/src/org/eclipse/jgit/lib/ObjectLoader.java
index 4839a1c28b..1a8d1ba9b2 100644
--- a/org.eclipse.jgit/src/org/eclipse/jgit/lib/ObjectLoader.java
+++ b/org.eclipse.jgit/src/org/eclipse/jgit/lib/ObjectLoader.java
@@ -89,17 +89,4 @@ public abstract class ObjectLoader {
 	 * @return the cached bytes of this object. Do not modify it.
 	 */
 	public abstract byte[] getCachedBytes();
-
-	/**
-	 * @return raw object type from object header, as stored in storage (pack,
-	 *         loose file). This may be different from {@link #getType()} result
-	 *         for packs (see {@link Constants}).
-	 */
-	public abstract int getRawType();
-
-	/**
-	 * @return raw size of object from object header (pack, loose file).
-	 *         Interpretation of this value depends on {@link #getRawType()}.
-	 */
-	public abstract long getRawSize();
 }
diff --git a/org.eclipse.jgit/src/org/eclipse/jgit/storage/file/DeltaOfsPackedObjectLoader.java b/org.eclipse.jgit/src/org/eclipse/jgit/storage/file/DeltaOfsPackedObjectLoader.java
index 1a5fe8ea90..059912a9cf 100644
--- a/org.eclipse.jgit/src/org/eclipse/jgit/storage/file/DeltaOfsPackedObjectLoader.java
+++ b/org.eclipse.jgit/src/org/eclipse/jgit/storage/file/DeltaOfsPackedObjectLoader.java
@@ -48,11 +48,6 @@ package org.eclipse.jgit.storage.file;
 
 import java.io.IOException;
 
-import org.eclipse.jgit.JGitText;
-import org.eclipse.jgit.errors.CorruptObjectException;
-import org.eclipse.jgit.lib.Constants;
-import org.eclipse.jgit.lib.ObjectId;
-
 /** Reads a deltified object which uses an offset to find its base. */
 class DeltaOfsPackedObjectLoader extends DeltaPackedObjectLoader {
 	private final long deltaBase;
@@ -67,18 +62,4 @@ class DeltaOfsPackedObjectLoader extends DeltaPackedObjectLoader {
 			throws IOException {
 		return pack.resolveBase(curs, deltaBase);
 	}
-
-	@Override
-	public int getRawType() {
-		return Constants.OBJ_OFS_DELTA;
-	}
-
-	@Override
-	ObjectId getDeltaBase() throws IOException {
-		final ObjectId id = pack.findObjectForOffset(deltaBase);
-		if (id == null)
-			throw new CorruptObjectException(
-					JGitText.get().offsetWrittenDeltaBaseForObjectNotFoundInAPack);
-		return id;
-	}
 }
diff --git a/org.eclipse.jgit/src/org/eclipse/jgit/storage/file/DeltaPackedObjectLoader.java b/org.eclipse.jgit/src/org/eclipse/jgit/storage/file/DeltaPackedObjectLoader.java
index 4a9323cccf..135282589f 100644
--- a/org.eclipse.jgit/src/org/eclipse/jgit/storage/file/DeltaPackedObjectLoader.java
+++ b/org.eclipse.jgit/src/org/eclipse/jgit/storage/file/DeltaPackedObjectLoader.java
@@ -104,11 +104,6 @@ abstract class DeltaPackedObjectLoader extends PackedObjectLoader {
 		}
 	}
 
-	@Override
-	public long getRawSize() {
-		return deltaSize;
-	}
-
 	/**
 	 * @param curs
 	 *            temporary thread storage during data access.
diff --git a/org.eclipse.jgit/src/org/eclipse/jgit/storage/file/DeltaRefPackedObjectLoader.java b/org.eclipse.jgit/src/org/eclipse/jgit/storage/file/DeltaRefPackedObjectLoader.java
index 40e1975e94..741ac74385 100644
--- a/org.eclipse.jgit/src/org/eclipse/jgit/storage/file/DeltaRefPackedObjectLoader.java
+++ b/org.eclipse.jgit/src/org/eclipse/jgit/storage/file/DeltaRefPackedObjectLoader.java
@@ -49,7 +49,6 @@ package org.eclipse.jgit.storage.file;
 import java.io.IOException;
 
 import org.eclipse.jgit.errors.MissingObjectException;
-import org.eclipse.jgit.lib.Constants;
 import org.eclipse.jgit.lib.ObjectId;
 
 /** Reads a deltified object which uses an {@link ObjectId} to find its base. */
@@ -69,14 +68,4 @@ class DeltaRefPackedObjectLoader extends DeltaPackedObjectLoader {
 			throw new MissingObjectException(deltaBase, "delta base");
 		return or;
 	}
-
-	@Override
-	public int getRawType() {
-		return Constants.OBJ_REF_DELTA;
-	}
-
-	@Override
-	ObjectId getDeltaBase() throws IOException {
-		return deltaBase;
-	}
 }
diff --git a/org.eclipse.jgit/src/org/eclipse/jgit/storage/file/PackFile.java b/org.eclipse.jgit/src/org/eclipse/jgit/storage/file/PackFile.java
index 01db79dcd4..783422bf38 100644
--- a/org.eclipse.jgit/src/org/eclipse/jgit/storage/file/PackFile.java
+++ b/org.eclipse.jgit/src/org/eclipse/jgit/storage/file/PackFile.java
@@ -209,7 +209,7 @@ public class PackFile implements Iterable<PackIndex.MutableEntry> {
 	 * @throws IOException
 	 *             the pack file or the index could not be read.
 	 */
-	public PackedObjectLoader get(final WindowCursor curs, final AnyObjectId id)
+	PackedObjectLoader get(final WindowCursor curs, final AnyObjectId id)
 			throws IOException {
 		final long offset = idx().findOffset(id);
 		return 0 < offset && !isCorrupt(offset) ? reader(curs, offset) : null;
diff --git a/org.eclipse.jgit/src/org/eclipse/jgit/storage/file/PackedObjectLoader.java b/org.eclipse.jgit/src/org/eclipse/jgit/storage/file/PackedObjectLoader.java
index e784f3d181..b575781cf8 100644
--- a/org.eclipse.jgit/src/org/eclipse/jgit/storage/file/PackedObjectLoader.java
+++ b/org.eclipse.jgit/src/org/eclipse/jgit/storage/file/PackedObjectLoader.java
@@ -48,7 +48,6 @@ package org.eclipse.jgit.storage.file;
 
 import java.io.IOException;
 
-import org.eclipse.jgit.lib.ObjectId;
 import org.eclipse.jgit.lib.ObjectLoader;
 
 /**
@@ -86,8 +85,6 @@ abstract class PackedObjectLoader extends ObjectLoader {
 	 * <li>{@link #getType()}</li>
 	 * <li>{@link #getSize()}</li>
 	 * <li>{@link #getBytes()}, {@link #getCachedBytes}</li>
-	 * <li>{@link #getRawSize()}</li>
-	 * <li>{@link #getRawType()}</li>
 	 * </ul>
 	 *
 	 * @param curs
@@ -109,19 +106,4 @@ abstract class PackedObjectLoader extends ObjectLoader {
 	public final byte[] getCachedBytes() {
 		return cachedBytes;
 	}
-
-	/**
-	 * @return offset of object header within pack file
-	 */
-	final long getObjectOffset() {
-		return objectOffset;
-	}
-
-	/**
-	 * @return id of delta base object for this object representation. null if
-	 *         object is not stored as delta.
-	 * @throws IOException
-	 *             when delta base cannot read.
-	 */
-	abstract ObjectId getDeltaBase() throws IOException;
 }
diff --git a/org.eclipse.jgit/src/org/eclipse/jgit/storage/file/UnpackedObjectLoader.java b/org.eclipse.jgit/src/org/eclipse/jgit/storage/file/UnpackedObjectLoader.java
index 054a4ae37e..b85ec149e9 100644
--- a/org.eclipse.jgit/src/org/eclipse/jgit/storage/file/UnpackedObjectLoader.java
+++ b/org.eclipse.jgit/src/org/eclipse/jgit/storage/file/UnpackedObjectLoader.java
@@ -214,14 +214,4 @@ public class UnpackedObjectLoader extends ObjectLoader {
 	public byte[] getCachedBytes() {
 		return bytes;
 	}
-
-	@Override
-	public int getRawType() {
-		return objectType;
-	}
-
-	@Override
-	public long getRawSize() {
-		return objectSize;
-	}
 }
diff --git a/org.eclipse.jgit/src/org/eclipse/jgit/storage/file/WholePackedObjectLoader.java b/org.eclipse.jgit/src/org/eclipse/jgit/storage/file/WholePackedObjectLoader.java
index ba7781a0f1..70fa616f73 100644
--- a/org.eclipse.jgit/src/org/eclipse/jgit/storage/file/WholePackedObjectLoader.java
+++ b/org.eclipse.jgit/src/org/eclipse/jgit/storage/file/WholePackedObjectLoader.java
@@ -52,7 +52,6 @@ import java.util.zip.DataFormatException;
 import org.eclipse.jgit.JGitText;
 import org.eclipse.jgit.errors.CorruptObjectException;
 import org.eclipse.jgit.lib.Constants;
-import org.eclipse.jgit.lib.ObjectId;
 
 /** Reader for a non-delta (just deflated) object in a pack file. */
 class WholePackedObjectLoader extends PackedObjectLoader {
@@ -94,19 +93,4 @@ class WholePackedObjectLoader extends PackedObjectLoader {
 			throw coe;
 		}
 	}
-
-	@Override
-	public int getRawType() {
-		return objectType;
-	}
-
-	@Override
-	public long getRawSize() {
-		return objectSize;
-	}
-
-	@Override
-	ObjectId getDeltaBase() {
-		return null;
-	}
 }