You are viewing a plain text version of this content. The canonical link for it is here.

Posted to notifications@groovy.apache.org by GitBox <gi...@apache.org> on 2021/06/06 21:05:43 UTC

[GitHub] [groovy] eric-milles commented on pull request #1591: GROOVY-10099 Behave when varargs method called with null single argument.

eric-milles commented on pull request #1591:
URL: https://github.com/apache/groovy/pull/1591#issuecomment-855462858


   There is a lot of discussion under GROOVY-10099 to unpack.  And then this commit presents quite a bit more in the comments and hundreds of lines of code to sift through.  Is it possible to break this down into smaller parts?  Like is there one test case that is the biggest source of confusion and is there a small change that can make a non-breaking improvement?
   
   I just don't see how we could accept a change of this size when the issue ticket itself was in dispute.  Best to pick up a smaller, non-controversial ticket to start with.


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

For queries about this service, please contact Infrastructure at:
users@infra.apache.org