This is an automated email from the ASF dual-hosted git repository.
He-Pin pushed a commit to branch main
in repository https://gitbox.apache.org/repos/asf/pekko.git
The following commit(s) were added to refs/heads/main by this push:
new fe5771080c refactor: use String.formatted() instead of String.format()
(#3144)
fe5771080c is described below
commit fe5771080c3243d040e3a90201b80f2aa98cd1ce
Author: He-Pin(kerr) <[email protected]>
AuthorDate: Tue Jun 23 17:17:47 2026 +0800
refactor: use String.formatted() instead of String.format() (#3144)
Motivation:
String.format(fmt, args) is a static method call. JDK 15 added
String.formatted(args) as a fluent instance method on the format string.
Modification:
Replace 2 occurrences of String.format(fmt, args) with fmt.formatted(args):
- UnsynchronizedByteArrayInputStream.java: range check error message
- AeronErrorLog.java: Aeron error logging
Result:
More readable fluent API style, functionally identical.
Tests:
sbt "actor/compile" "remote/compile" — passed
References:
Refs #3136
---
.../apache/pekko/io/UnsynchronizedByteArrayInputStream.java | 2 +-
.../org/apache/pekko/remote/artery/aeron/AeronErrorLog.java | 12 ++++++------
2 files changed, 7 insertions(+), 7 deletions(-)
diff --git
a/actor/src/main/java/org/apache/pekko/io/UnsynchronizedByteArrayInputStream.java
b/actor/src/main/java/org/apache/pekko/io/UnsynchronizedByteArrayInputStream.java
index 3ac6b0cc58..743eb303db 100644
---
a/actor/src/main/java/org/apache/pekko/io/UnsynchronizedByteArrayInputStream.java
+++
b/actor/src/main/java/org/apache/pekko/io/UnsynchronizedByteArrayInputStream.java
@@ -54,7 +54,7 @@ public class UnsynchronizedByteArrayInputStream extends
InputStream {
final int arrayLength = Objects.requireNonNull(array, "byte array").length;
if ((off | len | arrayLength) < 0 || arrayLength - len < off) {
throw new IndexOutOfBoundsException(
- String.format("Range [%s, %<s + %s) out of bounds for length %s",
off, len, arrayLength));
+ "Range [%s, %<s + %s) out of bounds for length %s".formatted(off,
len, arrayLength));
}
}
diff --git
a/remote/src/main/java/org/apache/pekko/remote/artery/aeron/AeronErrorLog.java
b/remote/src/main/java/org/apache/pekko/remote/artery/aeron/AeronErrorLog.java
index b4e5d78d07..ddd0c1b3da 100644
---
a/remote/src/main/java/org/apache/pekko/remote/artery/aeron/AeronErrorLog.java
+++
b/remote/src/main/java/org/apache/pekko/remote/artery/aeron/AeronErrorLog.java
@@ -65,12 +65,12 @@ public class AeronErrorLog {
lastObservationTimestamp,
encodedException) -> {
log.error(
- String.format(
- "Aeron error: %d observations from %s to %s for:%n %s",
- observationCount,
- Helpers.timestamp(firstObservationTimestamp),
- Helpers.timestamp(lastObservationTimestamp),
- encodedException));
+ "Aeron error: %d observations from %s to %s for:%n %s"
+ .formatted(
+ observationCount,
+ Helpers.timestamp(firstObservationTimestamp),
+ Helpers.timestamp(lastObservationTimestamp),
+ encodedException));
lastTimestamp.set(Math.max(lastTimestamp.get(),
lastObservationTimestamp));
},
sinceTimestamp);
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]