avro-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Doug Cutting (JIRA)" <j...@apache.org>
Subject [jira] Resolved: (AVRO-82) A large number of Java warnings due to unused vairables etc.
Date Tue, 28 Jul 2009 18:52:14 GMT

     [ 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.


Mime
View raw message