mgroovy commented on a change in pull request #1660:
URL: https://github.com/apache/groovy/pull/1660#discussion_r762621067



##########
File path: src/main/java/groovy/lang/NamedValue.java
##########
@@ -0,0 +1,24 @@
+package groovy.lang;
+
+public class NamedValue<T> {

Review comment:
       (Even if that is not for me to decide: This feature has been moved from 
Groovy 2.5, to Groovy 3, then to Groovy 4. It has been in discussion for years 
now. Keeping it out of users' hands further, seems, quite frankly, ridiculous. 
   
   I am, as you will know from discussions on the Groovy mailing list, very 
much for discussing changes to Groovy, weighing the pros and cons, trying to 
find benefits and pitfalls. I even sometimes take the opposite position to an 
argument which I feel is not strong enough to be for or against a proposal, 
even if I am myself not convinced what the right decision is, to stimulate 
further discussion, as I have done for the recent proposal regarding dates and 
dates-with-time objects.
   
   But in this case, even if you personally never log or println any variables, 
never need to embed them in exceptions, never need to generate any output where 
you need the name of variables or fields together with their value, I can, 
quite frankly not fathom what makes you continually fight this, in my eyes, 
small, harmless extension to Groovy so hard...)




-- 
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]


Reply via email to