honeymoose
/
OpenSearch
mirror of https://github.com/honeymoose/OpenSearch.git
1
0
Fork 0
Code Issues Packages Projects Releases Wiki Activity

Remove unnecessary call to MessageDigest.reset

This commit is contained in:
Jason Tedor 2015-10-04 21:02:50 -04:00
parent e5a10e9520
commit d3cef85352
2 changed files with 6 additions and 9 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

13
core/src/main/java/org/elasticsearch/common/hash/MessageDigests.java
View File

@ -41,23 +41,20 @@ public class MessageDigests {
}
public static MessageDigest md5() {
return cloneAndReset(MD5_DIGEST);
return clone(MD5_DIGEST);
}
public static MessageDigest sha1() {
return cloneAndReset(SHA_1_DIGEST);
return clone(SHA_1_DIGEST);
}
public static MessageDigest sha256() {
return cloneAndReset(SHA_256_DIGEST);
return clone(SHA_256_DIGEST);
}
private static MessageDigest cloneAndReset(MessageDigest messageDigest) {
private static MessageDigest clone(MessageDigest messageDigest) {
try {
MessageDigest clone = (MessageDigest) messageDigest.clone();
clone.reset();
return clone;
return (MessageDigest) messageDigest.clone();
} catch (CloneNotSupportedException e) {
throw new ElasticsearchException("Unexpected exception cloning MessageDigest instance", e);
}

2
core/src/test/java/org/elasticsearch/common/hash/MessageDigestsTests.java
View File

@ -70,7 +70,7 @@ public class MessageDigestsTests extends ESTestCase {
BigInteger expected = BigInteger.probablePrime(256, random());
byte[] bytes = expected.toByteArray();
String hex = MessageDigests.toHexString(bytes);
String zeros = new String(new char[bytes.length * 2]).replace("\0", "0");
String zeros = new String(new char[2 * bytes.length]).replace("\0", "0");
String expectedAsString = expected.toString(16);
String expectedHex = zeros.substring(expectedAsString.length()) + expectedAsString;
assertEquals(expectedHex, hex);