[GitHub] [commons-rng] aherbert commented on pull request #81: Minor Improvement:

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

aherbert commented on pull request #81:
URL: https://github.com/apache/commons-rng/pull/81#issuecomment-762250958


   The change to use final method arguments is a style change. We already use code style checks using the PMD rule `AvoidReassigningParameters` to ensure method arguments are effectively final. The gratuitous use of final does not add functionality to the code and in keeping with the source code for the JDK we have chosen not to add final in front of everything in the source.
   
   Also note that using an IDE to refactor all the code to add final has now broken code style formatting (see checkstyle failures on Travis) due to line length changes.
   
   I would remove all the use of final from all method arguments as this is already enforced by build checks. Then it is easy to see in the PR what is left to make final as local or instance variables.


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