ppkarwasz commented on code in PR #3645:
URL: https://github.com/apache/logging-log4j2/pull/3645#discussion_r2073521659
##########
log4j-api/src/main/java/org/apache/logging/log4j/message/ParameterFormatter.java:
##########
@@ -643,4 +643,212 @@ static String identityToString(final Object obj) {
}
return obj.getClass().getName() + '@' +
Integer.toHexString(System.identityHashCode(obj));
}
+
+ /**
+ * Adds a string representation of the contents of the specified array to
the buffer.
+ *
+ * @param a the array to convert (not null)
+ * @param str the {@code StringBuilder} that {@code a} will be appended
to
+ *
+ * @see Arrays#toString(byte[])
+ */
+ private static void appendArray(final byte[] a, final StringBuilder str) {
+ int iMax = a.length - 1;
+ if (iMax == -1) {
+ str.append("[]");
+ return;
+ }
+
+ str.append('[');
+ for (int i = 0; ; i++) {
+ str.append(a[i]);
+ if (i == iMax) {
+ str.append(']');
+ return;
+ }
+ str.append(", ");
+ }
+ }
+
+ /**
+ * Adds a string representation of the contents of the specified array to
the buffer.
+ *
+ * @param a the array to convert (not null)
+ * @param str the {@code StringBuilder} that {@code a} will be appended
to
+ *
+ * @see Arrays#toString(short[])
+ */
+ private static void appendArray(final short[] a, final StringBuilder str) {
+ int iMax = a.length - 1;
+ if (iMax == -1) {
+ str.append("[]");
+ return;
+ }
+
+ str.append('[');
+ for (int i = 0; ; i++) {
+ str.append(a[i]);
+ if (i == iMax) {
+ str.append(']');
+ return;
+ }
+ str.append(", ");
+ }
+ }
+
+ /**
+ * Adds a string representation of the contents of the specified array to
the buffer.
+ *
+ * @param a the array to convert (not null)
+ * @param str the {@code StringBuilder} that {@code a} will be appended
to
+ *
+ * @see Arrays#toString(int[])
+ */
+ private static void appendArray(final int[] a, final StringBuilder str) {
+ int iMax = a.length - 1;
+ if (iMax == -1) {
+ str.append("[]");
+ return;
+ }
+
+ str.append('[');
+ for (int i = 0; ; i++) {
+ str.append(a[i]);
+ if (i == iMax) {
+ str.append(']');
+ return;
+ }
+ str.append(", ");
+ }
+ }
Review Comment:
Please, check the statement about performance. I tested using Temurin 17 on
a Ryzen 9 processor, your mileage might of course vary.
--
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]
