[jira] Resolved: (AVRO-82) A large number of Java warnings due to unused vairables etc.

["Doug Cutting (JIRA)" <ji...@apache.org>]

     [ https://issues.apache.org/jira/browse/AVRO-82?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Doug Cutting resolved AVRO-82.
------------------------------

       Resolution: Fixed
    Fix Version/s: 1.0.1

I just committed this with a few changes:
 - added a few more checks;
 - made it so that so unit tests will not pass if checkstyle fails;
 - since checkstyle is LGPL, we cannot bundle it.  I improved our use of Ivy so that code that's only required to run tests is not bundled.  This also improves dependencies in our generated Maven POM file.

> The main reason for me create this JIRA is to avoid the large number of warnings I see in Eclipse.

Can you change your Eclipse settings, so that its warnings match those of checkstyle?  Perhaps we could even include an appropriate Eclipse settings file with the project.

Thanks, Thiru!

> A large number of Java warnings due to unused vairables etc.
> ------------------------------------------------------------
>
>                 Key: AVRO-82
>                 URL: https://issues.apache.org/jira/browse/AVRO-82
>             Project: Avro
>          Issue Type: Improvement
>          Components: java
>            Reporter: Thiruvalluvan M. G.
>            Assignee: Thiruvalluvan M. G.
>            Priority: Minor
>             Fix For: 1.0.1
>
>         Attachments: AVRO-82.patch, AVRO-82.patch
>
>
> I', creating this JIRA to submit a patch that fix the following:
>    - Unused imports
>    - Unused private fields, local variables
>    - Use of generic types without parameters
>    - Missing setialVersionUID for serializable classes
> There are more than 50 compiler warnings due to these.

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.