You are viewing a plain text version of this content. The canonical link for it is here.

Posted to dev@calcite.apache.org by "Vladimir Sitnikov (Jira)" <ji...@apache.org> on 2020/11/30 09:00:09 UTC

[jira] [Created] (CALCITE-4422) Add MethodCanBeStatic check via ErrorProne

Vladimir Sitnikov created CALCITE-4422:
------------------------------------------

             Summary: Add MethodCanBeStatic check via ErrorProne
                 Key: CALCITE-4422
                 URL: https://issues.apache.org/jira/browse/CALCITE-4422
             Project: Calcite
          Issue Type: Improvement
    Affects Versions: 1.26.0
            Reporter: Vladimir Sitnikov


See https://errorprone.info/bugpattern/MethodCanBeStatic

{quote}A private method that does not reference the enclosing instance can be static{quote}

Note: the key driver is code readability. If a method is static, it is clear that it can't access instance fields, and all it can access is provided via arguments.






--
This message was sent by Atlassian Jira
(v8.3.4#803005)