kinow commented on pull request #5:
URL: https://github.com/apache/commons-graph/pull/5#issuecomment-748590807
> IMO on type ensure that the reference will never change. For the values
types it will ensure that the value cannot change. If you check some of the
other apache commons modules you can see that they follow the same approach.
Yup, there are many modules in Commons that use `final`. I use it to ensure
immutability in some objects. But not everyone writes
```java
public void methodA(final param1, final param2) {
return param1 + param2;
}
```
Some projects may enforce it. But as far as I am aware, we have nothing in
Commons, nor in Commons Graph. So reviewers would have to inspect each PR to
ensure the code was consistent in using the `final` modifiers.
Otherwise, in a few months/years, the code would have a few methods and
local variables without the `final`, and we would need to keep updating the
code again, chasing for variables missing the `final` modifier.
Let's leave it open and see if others prefer to use the `final` modifier
everywhere it's possible in Graph (ping @garydgregory ?).
-Bruno
----------------------------------------------------------------
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:
us...@infra.apache.org
