ref(core): Drop redundant StringBuilder in hashing helper (JAVA-554)

runningcode · claude · runningcode · commit c248e08b1014 · 2026-06-10T09:20:54.000+02:00
Return the hex string directly instead of wrapping it in a StringBuilder
only to immediately call toString().

Co-Authored-By: Claude Opus 4.8 &lt;noreply@anthropic.com&gt;
diff --git a/sentry/src/main/java/io/sentry/util/StringUtils.java b/sentry/src/main/java/io/sentry/util/StringUtils.java
@@ -143,10 +143,9 @@ private StringUtils() {}
       final BigInteger no = new BigInteger(1, messageDigest);
 
       // Convert message digest into hex value
-      final StringBuilder stringBuilder = new StringBuilder(no.toString(16));
 
       // return the HashText
-      return stringBuilder.toString();
+      return no.toString(16);
     }
 
     // For specifying wrong message digest algorithms

Original file line number	Diff line number	Diff line change
`@@ -143,10 +143,9 @@ private StringUtils() {}`
`143`	`143`	`final BigInteger no = new BigInteger(1, messageDigest);`
`144`	`144`
`145`	`145`	`// Convert message digest into hex value`
`146`		`- final StringBuilder stringBuilder = new StringBuilder(no.toString(16));`
`147`	`146`
`148`	`147`	`// return the HashText`
`149`		`- return stringBuilder.toString();`
	`148`	`+ return no.toString(16);`
`150`	`149`	`}`
`151`	`150`
`152`	`151`	`// For specifying wrong message digest algorithms`