]> source.dussan.org Git - jgit.git/commitdiff
Fix public API issues introduced in I1baeedcc6946. 72/49872/3
authorYuxuan 'fishy' Wang <fishywang@google.com>
Wed, 10 Jun 2015 00:07:55 +0000 (17:07 -0700)
committerYuxuan 'fishy' Wang <fishywang@google.com>
Wed, 10 Jun 2015 00:20:13 +0000 (17:20 -0700)
Move ObjectCountCallback and WriteAbortedException to package
org.eclipse.jgit.transport, so that they'll become public API.

Change-Id: I95e3cfaa49f3f7371e794d5c253cf6981f87cae0
Signed-off-by: Yuxuan 'fishy' Wang <fishywang@google.com>
org.eclipse.jgit.test/tst/org/eclipse/jgit/transport/BundleWriterTest.java
org.eclipse.jgit/src/org/eclipse/jgit/internal/storage/pack/PackWriter.java
org.eclipse.jgit/src/org/eclipse/jgit/internal/storage/pack/WriteAbortedException.java [deleted file]
org.eclipse.jgit/src/org/eclipse/jgit/transport/BundleWriter.java
org.eclipse.jgit/src/org/eclipse/jgit/transport/ObjectCountCallback.java [new file with mode: 0644]
org.eclipse.jgit/src/org/eclipse/jgit/transport/WriteAbortedException.java [new file with mode: 0644]

index bd1ec33fa6a87a66bdae276a27b24c8745af0525..ba89d2d616d6832f6987f068f70860c4993ca5c2 100644 (file)
@@ -61,8 +61,6 @@ import java.util.Set;
 import org.eclipse.jgit.errors.MissingBundlePrerequisiteException;
 import org.eclipse.jgit.errors.NotSupportedException;
 import org.eclipse.jgit.errors.TransportException;
-import org.eclipse.jgit.internal.storage.pack.PackWriter.ObjectCountCallback;
-import org.eclipse.jgit.internal.storage.pack.WriteAbortedException;
 import org.eclipse.jgit.lib.NullProgressMonitor;
 import org.eclipse.jgit.lib.ObjectId;
 import org.eclipse.jgit.lib.Ref;
index 74016027250b05c4738c675be8dfcad4add62680..af02f4ceda2d565948399bd2ee313d4811a10888 100644 (file)
@@ -114,6 +114,8 @@ import org.eclipse.jgit.revwalk.RevSort;
 import org.eclipse.jgit.revwalk.RevTag;
 import org.eclipse.jgit.revwalk.RevTree;
 import org.eclipse.jgit.storage.pack.PackConfig;
+import org.eclipse.jgit.transport.ObjectCountCallback;
+import org.eclipse.jgit.transport.WriteAbortedException;
 import org.eclipse.jgit.util.BlockList;
 import org.eclipse.jgit.util.TemporaryBuffer;
 
@@ -167,31 +169,6 @@ public class PackWriter implements AutoCloseable {
                boolean contains(AnyObjectId objectId);
        }
 
-       /**
-        * A callback to tell caller the count of objects ASAP.
-        *
-        * @since 4.1
-        */
-       public interface ObjectCountCallback {
-               /**
-                * Invoked when the PackWriter has counted the objects to be written
-                * to pack.
-                * <p>
-                * An ObjectCountCallback can use this information to decide whether
-                * the
-                * {@link #writePack(ProgressMonitor, ProgressMonitor, OutputStream)}
-                * operation should be aborted.
-                * <p>
-                * This callback will be called exactly once.
-                *
-                * @param objectCount
-                *            the count of the objects.
-                * @throws WriteAbortedException
-                *             to indicate that the write operation should be aborted.
-                */
-               void setObjectCount(long objectCount) throws WriteAbortedException;
-       }
-
        private static final Map<WeakReference<PackWriter>, Boolean> instances =
                        new ConcurrentHashMap<WeakReference<PackWriter>, Boolean>();
 
@@ -952,7 +929,7 @@ public class PackWriter implements AutoCloseable {
         *             stream.
         * @throws WriteAbortedException
         *             the write operation is aborted by
-        *             {@link PackWriter.ObjectCountCallback}.
+        *             {@link ObjectCountCallback}.
         */
        public void writePack(ProgressMonitor compressMonitor,
                        ProgressMonitor writeMonitor, OutputStream packStream)
diff --git a/org.eclipse.jgit/src/org/eclipse/jgit/internal/storage/pack/WriteAbortedException.java b/org.eclipse.jgit/src/org/eclipse/jgit/internal/storage/pack/WriteAbortedException.java
deleted file mode 100644 (file)
index 1bb8743..0000000
+++ /dev/null
@@ -1,83 +0,0 @@
-/*
- * Copyright (C) 2015, Google Inc.
- * and other copyright owners as documented in the project's IP log.
- *
- * This program and the accompanying materials are made available
- * under the terms of the Eclipse Distribution License v1.0 which
- * accompanies this distribution, is reproduced below, and is
- * available at http://www.eclipse.org/org/documents/edl-v10.php
- *
- * All rights reserved.
- *
- * Redistribution and use in source and binary forms, with or
- * without modification, are permitted provided that the following
- * conditions are met:
- *
- * - Redistributions of source code must retain the above copyright
- *   notice, this list of conditions and the following disclaimer.
- *
- * - Redistributions in binary form must reproduce the above
- *   copyright notice, this list of conditions and the following
- *   disclaimer in the documentation and/or other materials provided
- *   with the distribution.
- *
- * - Neither the name of the Eclipse Foundation, Inc. nor the
- *   names of its contributors may be used to endorse or promote
- *   products derived from this software without specific prior
- *   written permission.
- *
- * THIS SOFTWARE IS PROVIDED BY THE COPYRIGHT HOLDERS AND
- * CONTRIBUTORS "AS IS" AND ANY EXPRESS OR IMPLIED WARRANTIES,
- * INCLUDING, BUT NOT LIMITED TO, THE IMPLIED WARRANTIES
- * OF MERCHANTABILITY AND FITNESS FOR A PARTICULAR PURPOSE
- * ARE DISCLAIMED. IN NO EVENT SHALL THE COPYRIGHT OWNER OR
- * CONTRIBUTORS BE LIABLE FOR ANY DIRECT, INDIRECT, INCIDENTAL,
- * SPECIAL, EXEMPLARY, OR CONSEQUENTIAL DAMAGES (INCLUDING, BUT
- * NOT LIMITED TO, PROCUREMENT OF SUBSTITUTE GOODS OR SERVICES;
- * LOSS OF USE, DATA, OR PROFITS; OR BUSINESS INTERRUPTION) HOWEVER
- * CAUSED AND ON ANY THEORY OF LIABILITY, WHETHER IN CONTRACT,
- * STRICT LIABILITY, OR TORT (INCLUDING NEGLIGENCE OR OTHERWISE)
- * ARISING IN ANY WAY OUT OF THE USE OF THIS SOFTWARE, EVEN IF
- * ADVISED OF THE POSSIBILITY OF SUCH DAMAGE.
- */
-
-package org.eclipse.jgit.internal.storage.pack;
-
-import java.io.IOException;
-
-/**
- * An exception to be thrown when the write operation is aborted.
- * <p>
- * That can be thrown inside
- * {@link PackWriter.ObjectCountCallback#setObjectCount(long)}.
- *
- * @since 4.1
- */
-public class WriteAbortedException extends IOException {
-       /**
-        * Construct a {@code WriteAbortedException}.
-        */
-       public WriteAbortedException() {
-       }
-
-       /**
-        * Construct a {@code WriteAbortedException}.
-        *
-        * @param s message describing the issue
-        */
-       public WriteAbortedException(String s) {
-               super(s);
-       }
-
-       /**
-        * Construct a {@code WriteAbortedException}.
-        *
-        * @param s
-        *            message describing the issue
-        * @param why
-        *            a lower level implementation specific issue.
-        */
-       public WriteAbortedException(String s, Throwable why) {
-               super(s, why);
-       }
-}
index 1e7aff88c33bd53bfdec906f47d1d263cbed876f..ca624c03d5ffb1b6d03f36b415a93c4ab84cea51 100644 (file)
@@ -55,8 +55,6 @@ import java.util.TreeMap;
 
 import org.eclipse.jgit.internal.JGitText;
 import org.eclipse.jgit.internal.storage.pack.PackWriter;
-import org.eclipse.jgit.internal.storage.pack.PackWriter.ObjectCountCallback;
-import org.eclipse.jgit.internal.storage.pack.WriteAbortedException;
 import org.eclipse.jgit.lib.AnyObjectId;
 import org.eclipse.jgit.lib.Constants;
 import org.eclipse.jgit.lib.ObjectId;
diff --git a/org.eclipse.jgit/src/org/eclipse/jgit/transport/ObjectCountCallback.java b/org.eclipse.jgit/src/org/eclipse/jgit/transport/ObjectCountCallback.java
new file mode 100644 (file)
index 0000000..ee50a25
--- /dev/null
@@ -0,0 +1,71 @@
+/*
+ * Copyright (C) 2015, Google Inc.
+ * and other copyright owners as documented in the project's IP log.
+ *
+ * This program and the accompanying materials are made available
+ * under the terms of the Eclipse Distribution License v1.0 which
+ * accompanies this distribution, is reproduced below, and is
+ * available at http://www.eclipse.org/org/documents/edl-v10.php
+ *
+ * All rights reserved.
+ *
+ * Redistribution and use in source and binary forms, with or
+ * without modification, are permitted provided that the following
+ * conditions are met:
+ *
+ * - Redistributions of source code must retain the above copyright
+ *   notice, this list of conditions and the following disclaimer.
+ *
+ * - Redistributions in binary form must reproduce the above
+ *   copyright notice, this list of conditions and the following
+ *   disclaimer in the documentation and/or other materials provided
+ *   with the distribution.
+ *
+ * - Neither the name of the Eclipse Foundation, Inc. nor the
+ *   names of its contributors may be used to endorse or promote
+ *   products derived from this software without specific prior
+ *   written permission.
+ *
+ * THIS SOFTWARE IS PROVIDED BY THE COPYRIGHT HOLDERS AND
+ * CONTRIBUTORS "AS IS" AND ANY EXPRESS OR IMPLIED WARRANTIES,
+ * INCLUDING, BUT NOT LIMITED TO, THE IMPLIED WARRANTIES
+ * OF MERCHANTABILITY AND FITNESS FOR A PARTICULAR PURPOSE
+ * ARE DISCLAIMED. IN NO EVENT SHALL THE COPYRIGHT OWNER OR
+ * CONTRIBUTORS BE LIABLE FOR ANY DIRECT, INDIRECT, INCIDENTAL,
+ * SPECIAL, EXEMPLARY, OR CONSEQUENTIAL DAMAGES (INCLUDING, BUT
+ * NOT LIMITED TO, PROCUREMENT OF SUBSTITUTE GOODS OR SERVICES;
+ * LOSS OF USE, DATA, OR PROFITS; OR BUSINESS INTERRUPTION) HOWEVER
+ * CAUSED AND ON ANY THEORY OF LIABILITY, WHETHER IN CONTRACT,
+ * STRICT LIABILITY, OR TORT (INCLUDING NEGLIGENCE OR OTHERWISE)
+ * ARISING IN ANY WAY OUT OF THE USE OF THIS SOFTWARE, EVEN IF
+ * ADVISED OF THE POSSIBILITY OF SUCH DAMAGE.
+ */
+
+package org.eclipse.jgit.transport;
+
+import org.eclipse.jgit.internal.storage.pack.PackWriter;
+
+/**
+ * A callback to tell caller the count of objects ASAP.
+ *
+ * @since 4.1
+ */
+public interface ObjectCountCallback {
+       /**
+        * Invoked when the {@link PackWriter} has counted the objects to be
+        * written to pack.
+        * <p>
+        * An {@code ObjectCountCallback} can use this information to decide
+        * whether the
+        * {@link PackWriter#writePack(ProgressMonitor, ProgressMonitor, OutputStream)}
+        * operation should be aborted.
+        * <p>
+        * This callback will be called exactly once.
+        *
+        * @param objectCount
+        *            the count of the objects.
+        * @throws WriteAbortedException
+        *             to indicate that the write operation should be aborted.
+        */
+       void setObjectCount(long objectCount) throws WriteAbortedException;
+}
diff --git a/org.eclipse.jgit/src/org/eclipse/jgit/transport/WriteAbortedException.java b/org.eclipse.jgit/src/org/eclipse/jgit/transport/WriteAbortedException.java
new file mode 100644 (file)
index 0000000..5ba4893
--- /dev/null
@@ -0,0 +1,83 @@
+/*
+ * Copyright (C) 2015, Google Inc.
+ * and other copyright owners as documented in the project's IP log.
+ *
+ * This program and the accompanying materials are made available
+ * under the terms of the Eclipse Distribution License v1.0 which
+ * accompanies this distribution, is reproduced below, and is
+ * available at http://www.eclipse.org/org/documents/edl-v10.php
+ *
+ * All rights reserved.
+ *
+ * Redistribution and use in source and binary forms, with or
+ * without modification, are permitted provided that the following
+ * conditions are met:
+ *
+ * - Redistributions of source code must retain the above copyright
+ *   notice, this list of conditions and the following disclaimer.
+ *
+ * - Redistributions in binary form must reproduce the above
+ *   copyright notice, this list of conditions and the following
+ *   disclaimer in the documentation and/or other materials provided
+ *   with the distribution.
+ *
+ * - Neither the name of the Eclipse Foundation, Inc. nor the
+ *   names of its contributors may be used to endorse or promote
+ *   products derived from this software without specific prior
+ *   written permission.
+ *
+ * THIS SOFTWARE IS PROVIDED BY THE COPYRIGHT HOLDERS AND
+ * CONTRIBUTORS "AS IS" AND ANY EXPRESS OR IMPLIED WARRANTIES,
+ * INCLUDING, BUT NOT LIMITED TO, THE IMPLIED WARRANTIES
+ * OF MERCHANTABILITY AND FITNESS FOR A PARTICULAR PURPOSE
+ * ARE DISCLAIMED. IN NO EVENT SHALL THE COPYRIGHT OWNER OR
+ * CONTRIBUTORS BE LIABLE FOR ANY DIRECT, INDIRECT, INCIDENTAL,
+ * SPECIAL, EXEMPLARY, OR CONSEQUENTIAL DAMAGES (INCLUDING, BUT
+ * NOT LIMITED TO, PROCUREMENT OF SUBSTITUTE GOODS OR SERVICES;
+ * LOSS OF USE, DATA, OR PROFITS; OR BUSINESS INTERRUPTION) HOWEVER
+ * CAUSED AND ON ANY THEORY OF LIABILITY, WHETHER IN CONTRACT,
+ * STRICT LIABILITY, OR TORT (INCLUDING NEGLIGENCE OR OTHERWISE)
+ * ARISING IN ANY WAY OUT OF THE USE OF THIS SOFTWARE, EVEN IF
+ * ADVISED OF THE POSSIBILITY OF SUCH DAMAGE.
+ */
+
+package org.eclipse.jgit.transport;
+
+import java.io.IOException;
+
+/**
+ * An exception to be thrown when the write operation is aborted.
+ * <p>
+ * That can be thrown inside
+ * {@link ObjectCountCallback#setObjectCount(long)}.
+ *
+ * @since 4.1
+ */
+public class WriteAbortedException extends IOException {
+       /**
+        * Construct a {@code WriteAbortedException}.
+        */
+       public WriteAbortedException() {
+       }
+
+       /**
+        * Construct a {@code WriteAbortedException}.
+        *
+        * @param s message describing the issue
+        */
+       public WriteAbortedException(String s) {
+               super(s);
+       }
+
+       /**
+        * Construct a {@code WriteAbortedException}.
+        *
+        * @param s
+        *            message describing the issue
+        * @param why
+        *            a lower level implementation specific issue.
+        */
+       public WriteAbortedException(String s, Throwable why) {
+               super(s, why);
+       }
+}