nfsantos commented on code in PR #1599:
URL: https://github.com/apache/jackrabbit-oak/pull/1599#discussion_r1694770732
##########
oak-commons/src/main/java/org/apache/jackrabbit/oak/commons/LongUtils.java:
##########
@@ -51,4 +52,83 @@ public static long safeAdd(long a, long b) {
public static long calculateExpirationTime(long expiration) {
return LongUtils.safeAdd(expiration, new Date().getTime());
}
+
+ /*
+ * Taken from
https://github.com/google/guava/blob/0c33dd12b193402cdf6962d43d69743521aa2f76/guava/src/com/google/common/primitives/Longs.java#L334
+ */
+ private static final class AsciiDigits {
+ private AsciiDigits() {
+ }
+
+ private static final byte[] asciiDigits;
+
+ static {
+ byte[] result = new byte[128];
+ Arrays.fill(result, (byte) -1);
+ for (int i = 0; i < 10; i++) {
+ result['0' + i] = (byte) i;
+ }
+ for (int i = 0; i < 26; i++) {
+ result['A' + i] = (byte) (10 + i);
+ result['a' + i] = (byte) (10 + i);
+ }
+ asciiDigits = result;
+ }
+
+ static int digit(char c) {
+ return (c < 128) ? asciiDigits[c] : -1;
+ }
+ }
+
+ /**
+ * Parses the specified string as a signed decimal long value. Unlike
{@link Long#parseLong(String)}, this method
+ * returns {@code null} instead of throwing an exception if parsing fails.
This can be significantly more efficient
+ * when the string to be parsed is often invalid, as raising an exception
is an expensive operation.
+ *<p>
+ * This is a simplified version of
+ * <a
href="https://github.com/google/guava/blob/0c33dd12b193402cdf6962d43d69743521aa2f76/guava/src/com/google/common/primitives/Longs.java#L400";>Longs.tryParse()</a>
+ * in Guava. This version is hardcoded to only support radix 10.
+ * <p>
+ *
+ * @see
org.apache.jackrabbit.guava.common.primitives.Longs#tryParse(String)
+ */
+ public static Long tryParse(String string) {
+ if (string == null) {
+ return null;
+ }
+ int radix = 10;
+ boolean negative = string.charAt(0) == '-';
+ int index = negative ? 1 : 0;
+ if (index == string.length()) {
+ return null;
+ }
+ int digit = AsciiDigits.digit(string.charAt(index++));
+ if (digit < 0 || digit >= radix) {
+ return null;
+ }
+ long accum = -digit;
Review Comment:
I took the code as a black box, I think we do not need to fully understand
it, as long as we do not modify it. This is just a workaround to the
restrictions of using Guava directly.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
