[GitHub] [maven] rfscholte commented on issue #303: [wip] [MNG-6831] replace anonymous classes by lambdas

[GitBox <gi...@apache.org>]

rfscholte commented on issue #303: [wip] [MNG-6831] replace anonymous classes by lambdas
URL: https://github.com/apache/maven/pull/303#issuecomment-569235422
 
 
   Regarding the lambda's in general: for those that have been programming Java for over 15 years (like me) having anonymous classes feels more comfortable. Reading lambda's takes time for us, because in our minds we keep trying to read them with inner classes.
   For the generation that grew up with lambda's it is a nobrainer.
   This is probably a good moment to let go the comfortzone. We know that while Maven was on Java 7 we missed new contributors providing patches because they couldn't use these Java 8 features.
   
   Regarding refactoring in general: refactoring should be a constant factor. It is part of the process and IMO often doesn't deserve a separate issuenumber.

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


With regards,
Apache Git Services