mgroovy commented on a change in pull request #1660:
URL: https://github.com/apache/groovy/pull/1660#discussion_r762597151
##########
File path: src/main/java/groovy/lang/NamedValue.java
##########
@@ -0,0 +1,24 @@
+package groovy.lang;
+
+public class NamedValue<T> {
Review comment:
Map.Entry<String,T> & Tuple2<String,T>: No, both do not have
1. The desired toString behavior: The no core application for NV is logging
without having to write the typical "person=$pin, user=$user,
insertSql='$insertSql'" (the first term is an example of a typical refactoring
error here: the code was copy-pasted from another place where "person" was the
first var, and only the variable name was refactored, leading to a misleading
log output).
2. Speaking member access (name, val).
--
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]
