castor

Compiler warnings: 268 unused imports

Details

  • Type: Improvement Improvement
  • Status: Closed Closed
  • Priority: Minor Minor
  • Resolution: Fixed
  • Affects Version/s: None
  • Fix Version/s: 1.0.1
  • Component/s: General
  • Labels:
    None
  • Number of attachments :
    1

Description

Unused imports should be removed as they might introduce extra dependencies.

Issue Links

is depended upon by

Improvement - An improvement or enhancement to an existing feature or task. CASTOR-1400 Improve code quality by removing compiler warnings

  • Major - Major loss of function.
  • Closed - The issue is considered finished, the resolution is correct. Issues which are not closed can be reopened.

Activity

Ascending order - Click to sort in descending order
Hide
Permalink
Henk van Voorthujsen added a comment -

AFAIK, this removes all unnecessary imports that are not in generated files.

Show
Henk van Voorthujsen added a comment - AFAIK, this removes all unnecessary imports that are not in generated files.
Hide
Permalink
Werner Guttmann added a comment -

I actually wonder whether to apply the patch provided against classes stored in org.exolab.castor.maping.xml and org.exolab.castor.maping.xml.types, as these classes are generated.

Show
Werner Guttmann added a comment - I actually wonder whether to apply the patch provided against classes stored in org.exolab.castor.maping.xml and org.exolab.castor.maping.xml.types, as these classes are generated.
Hide
Permalink
Henk van Voorthujsen added a comment -

I was under the impression that Eclipse leaves out the changes that i remove from the synchronize view - obviously not.
I'll see if I can submit a more constrained patch.

Show
Henk van Voorthujsen added a comment - I was under the impression that Eclipse leaves out the changes that i remove from the synchronize view - obviously not. I'll see if I can submit a more constrained patch.
Hide
Permalink
Henk van Voorthujsen added a comment -

It turns out that Eclipse 3.2 has a filter option for creating patches. Attached a more constrained patch.

Show
Henk van Voorthujsen added a comment - It turns out that Eclipse 3.2 has a filter option for creating patches. Attached a more constrained patch.
Hide
Permalink
Ralf Joachim added a comment -

Will commit as is with only release notes added.

Show
Ralf Joachim added a comment - Will commit as is with only release notes added.

People

  • Assignee:
    Ralf Joachim
    Reporter:
    Henk van Voorthujsen
Vote (0)
Watch (0)

Dates

  • Created:
    Updated:
    Resolved: