RequireThisCheck doesn't check methods about checkstyle HOT 4 CLOSED

referencing method with "this" is very weird thing to my mind, Checkstyle could do it, but ...

1. Can you please extend description why you need this?
2. What is a problem you are try to resolve by means of this?
3. a bit more evidence that somebody require this too, and do use such notation.

there is no room for this feature to be implemented by us in nearest future, we are tiny team and in progress of SF->Github transition, pull request are welcome.
It have to be done a an option for this Check and it will be switched OFF by default for sure.

If no good evidence will be provided for this feature, and All admins give approval, you can host it in our experimental pool of Checks https://github.com/sevntu-checkstyle/sevntu.checkstyle/issues?state=open. in that pool we can host any Checks, to let you test Check before it prove it is in demand by wide community.

from checkstyle.

According to checkstyle documentation (at http://checkstyle.sourceforge.net/config_coding.html#RequireThis) it should already handle both fields and methods, I assume that at some point it did but later it just broke.
So no need to create a seperate check, just a fix to the current one (or a fix to documentation to remove the checkMethods whether we should check methods usage or not text)

from checkstyle.

Thanks a lot for point to the documentation, now issue is classified as bug, will be fixed soon, you are welcome with pull request.

from checkstyle.

My fix is at #42

from checkstyle.