I'm so glad somebody else gets bugged by all the trivial warnings, all along I thought it was a personal problem <G>..
As I remember, I deprecated LuceneTestCase entirely to encourage people to migrate to the Junit4 variant (LuceneTestCaseJ4). So removing those deprecations should be approached with some caution. Of course this may have changed in the interim.... Erick On Thu, Feb 25, 2010 at 10:01 AM, Shai Erera (JIRA) <j...@apache.org> wrote: > > [ > https://issues.apache.org/jira/browse/LUCENE-2285?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel] > > Shai Erera updated LUCENE-2285: > ------------------------------- > > Attachment: LUCENE-2285.patch > > Quite a large patch. I've started off with 3832 compiler warnings based on > my eclipse settings and we're now down to 510. All tests pass, including > core, contrib and tag. I've also fixed a bunch of javadocs warnings, and > "ant javadocs" now passes cleanly. I did not do any formatting to the code, > in order to preserve the patch as clear and focused as possible, even though > it's a very large one ... > > It touches a lot of files. So the sooner someone can help me commit it the > better (before these files change). > > > Code cleanup from all sorts of (trivial) warnings > > ------------------------------------------------- > > > > Key: LUCENE-2285 > > URL: https://issues.apache.org/jira/browse/LUCENE-2285 > > Project: Lucene - Java > > Issue Type: Improvement > > Reporter: Shai Erera > > Priority: Minor > > Fix For: 3.1 > > > > Attachments: LUCENE-2285.patch > > > > > > I would like to do some code cleanup and remove all sorts of trivial > warnings, like unnecessary casts, problems w/ javadocs, unused variables, > redundant null checks, unnecessary semicolon etc. These are all very trivial > and should not pose any problem. > > I'll create another issue for getting rid of deprecated code usage, like > LuceneTestCase and all sorts of deprecated constructors. That's also trivial > because it only affects Lucene code, but it's a different type of change. > > Another issue I'd like to create is about introducing more generics in > the code, where it's missing today - not changing existing API. There are > many places in the code like that. > > So, with you permission, I'll start with the trivial ones first, and then > move on to the others. > > -- > This message is automatically generated by JIRA. > - > You can reply to this email to add a comment to the issue online. > > > --------------------------------------------------------------------- > To unsubscribe, e-mail: java-dev-unsubscr...@lucene.apache.org > For additional commands, e-mail: java-dev-h...@lucene.apache.org > >
