Thanks for the clarification :)

On 28 Apr 2016 02:12, "John Wagenleitner" <> wrote:

On Sun, Apr 24, 2016 at 11:13 PM, Mario Garcia <> wrote:

Besides, I was wondering If most, if not all these static methods, should have all parameters marked as final. Is there any policy about this ? Would it help ?

I don't think there's a policy, personally I tend to not use final for local/parameters unless it's used in an anonymous inner class.  Hopefully the methods are short enough that the extra syntax is not needed to know if it reassigned or not. 

2016-04-24 21:46 GMT+02:00 Jochen Theodorou <>:
On 24.04.2016 18:12, John Wagenleitner wrote:
About to merge in PR 290 [1] and wanted to do a quick poll to see if
there were any objections since it touches quite a few files across core
and sub-modules.  Any objections to merging this into master?  And


I guess it is ok. I did see two package private methods made private instead of only private ones, but even those should be ok. so unless I did oversee something I do not really have any objection here.

bye Jochen