You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@thrift.apache.org by "Christopher Tubbs (JIRA)" <ji...@apache.org> on 2017/02/08 03:47:41 UTC

[jira] [Created] (THRIFT-4074) Unnecessary suppress warnings (unused)

Christopher Tubbs created THRIFT-4074:
-----------------------------------------

             Summary: Unnecessary suppress warnings (unused)
                 Key: THRIFT-4074
                 URL: https://issues.apache.org/jira/browse/THRIFT-4074
             Project: Thrift
          Issue Type: Bug
          Components: Java - Compiler
    Affects Versions: 0.10.0
            Reporter: Christopher Tubbs


Some generated Java files do not have unused variables. This results an a warning in some compilers (Eclipse, for one), that the warnings suppression for unused elements itself is unnecessary.

Some unused variables appear to simply be unnecessary code generation. These should be removed.

Others may affect serialization if changed, and are intentionally unused for backwards compatibility because they are class fields.
If this is the case, the warnings suppression about them being unused should appear on the unused item itself when it is declared, rather than on the class as a whole.

If these two steps are done, then there is no need to continue to suppress the unused types for all files, globally at the top of the file.




--
This message was sent by Atlassian JIRA
(v6.3.15#6346)