gemmellr commented on code in PR #5477:
URL: https://github.com/apache/activemq-artemis/pull/5477#discussion_r1934142083
##########
artemis-core-client/src/main/java/org/apache/activemq/artemis/api/core/JsonUtil.java:
##########
@@ -111,14 +108,13 @@ public static Object[] fromJsonArray(final JsonArray
jsonArray) throws Exception
}
array[i] = map;
- } else if (val instanceof JsonString) {
- array[i] = ((JsonString) val).getString();
+ } else if (val instanceof JsonString string) {
+ array[i] = string.getString();
Review Comment:
I'm giving up looking at the rest of the json stuff...same type of recent
comments+suggestions to make it more readable than originally, rather than
arguably less readable, also apply to the rest of it as well.
--
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]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
For further information, visit: https://activemq.apache.org/contact
