garydgregory commented on code in PR #1417:
URL: https://github.com/apache/commons-lang/pull/1417#discussion_r2217892944
##########
src/main/java/org/apache/commons/lang3/ArrayUtils.java:
##########
@@ -9525,42 +9525,43 @@ public static String toString(final Object array, final
String stringIfNull) {
return new ToStringBuilder(array,
ToStringStyle.SIMPLE_STYLE).append(array).toString();
}
- /**
- * Returns an array containing the string representation of each element
in the argument array.
- * <p>
- * This method returns {@code null} for a {@code null} input array.
- * </p>
- *
- * @param array the {@code Object[]} to be processed, may be {@code null}.
- * @return {@code String[]} of the same size as the source with its
element's string representation,
- * {@code null} if null array input
- * @since 3.6
- */
- public static String[] toStringArray(final Object[] array) {
- return toStringArray(array, "null");
+ /**
+ * Converts an array of objects to an array of strings, handling {@code null}
values gracefully.
Review Comment:
I think I am going towards -1 because this looks to me like change for the
sake of change. Why call it "gracefully"? Should we document when an
implementation is not garceful?
--
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]
