Remove Fingerprint.toString() to avoid relying on MessageDigest.clone().

--
MOS_MIGRATED_REVID=126566280

1 files changed, 1 insertions, 22 deletions

diff --git a/src/main/java/com/google/devtools/build/lib/util/Fingerprint.java b/src/main/java/com/google/devtools/build/lib/util/Fingerprint.java
index eb053f9be6..a461041c3d 100644
--- a/src/main/java/com/google/devtools/build/lib/util/Fingerprint.java
+++ b/src/main/java/com/google/devtools/build/lib/util/Fingerprint.java
@@ -28,8 +28,7 @@ import java.util.UUID;
 import javax.annotation.Nullable;
 
 /**
- * Simplified wrapper for MD5 message digests. See also
- * com.google.math.crypto.MD5HMAC for a similar interface.
+ * Simplified wrapper for MD5 message digests.
  *
  * @see java.security.MessageDigest
  */
@@ -67,7 +66,6 @@ public final class Fingerprint {
    * <p>This method has the side-effect of resetting the underlying digest computer.
    *
    * @return the MD5 digest as a 32-character string of hexadecimal digits
-   * @see com.google.math.crypto.MD5HMAC#toString()
    */
   public String hexDigestAndReset() {
     return hexDigest(digestAndReset());
@@ -90,25 +88,6 @@ public final class Fingerprint {
   }
 
   /**
-   * Override of Object.toString to return a string for the MD5 digest without
-   * finalizing the digest computation. Calling hexDigest() instead will
-   * finalize the digest computation.
-   *
-   * @return the string returned by hexDigest()
-   */
-  @Override
-  public String toString() {
-    try {
-      // MD5 does support cloning, so this should not fail
-      return hexDigest(((MessageDigest) md.clone()).digest());
-    } catch (CloneNotSupportedException e) {
-      // MessageDigest does not support cloning,
-      // so just return the toString() on the MessageDigest.
-      return md.toString();
-    }
-  }
-
-  /**
    * Updates the digest with 0 or more bytes.
    *
    * @param input the array of bytes with which to update the digest