PMD / Issues / #254 False+ : UnusedImport with Javadoc @throws

#254 False+ : UnusedImport with Javadoc @throws

Milestone: PMD-5.0.4

Status: closed

Owner: Andreas Dangel

Labels: None

Module: PMD

Priority: 3-Major

Type: Bug

Affects version: [4,

Ruleset / Rule:

Updated: 2014-04-30

Created: 2004-11-26

Creator: Florent Coste

Private: No

Javadoc generator uses import to fully qualified
classes mentionned in @link, for instance

@deprecated use {@link
ServiceProfileAdmin#setServiceStructure(ServiceProfileDescriptor,
ServiceStructureDescriptor)} instead.

PMD detect those imports as unused. (Eclipse behaves
correctly on this)

Discussion

Tom Copeland - 2004-11-29

Logged In: YES
user_id=5159

Hi Florent -

Hmmm...... very interesting. Since PMD discards comments,
it doesn't pick this up. Hm. I hesitate to call this a
bug... but it would be nice if it picked up this "usage", so
I suppose it is....

Yours,

Tom

If you would like to refer to this comment somewhere else in this project, copy and paste the following link:

Dmitry Malenok - 2012-03-20

The similar issue was fixed in checkstyle in 2011 (https://sourceforge.net/tracker/?func=detail&atid=397078&aid=652303&group_id=29721).

If you would like to refer to this comment somewhere else in this project, copy and paste the following link:

thomaslennon - 2013-02-22

hi
I'm interested in this "bug" due to hitting it in our project today. We were using pmd 4.6, but after downloading 5.0.2 I noticed it was still a problem. Can I help to fix it?

If you would like to refer to this comment somewhere else in this project, copy and paste the following link:

Andreas Dangel - 2013-03-02

assigned_to: Andreas Dangel

module: --> PMD

milestone: --> PMD-5.0.3

priority: 5 --> 3-Major

type: --> Bug

affects_version: --> [4,
If you would like to refer to this comment somewhere else in this project, copy and paste the following link:

Andreas Dangel - 2013-03-14

Hi,
can you provide me a sample file which triggers this false positive?
PMD should already properly handle @see, @link and @value tags in javadocs.
Thanks,
Andreas

If you would like to refer to this comment somewhere else in this project, copy and paste the following link:

Andreas Dangel - 2013-03-15

status: open --> more-info-needed
If you would like to refer to this comment somewhere else in this project, copy and paste the following link:

Andreas Dangel - 2013-04-05

status: more-info-needed --> not-reproducible
If you would like to refer to this comment somewhere else in this project, copy and paste the following link:

thomaslennon - 2013-04-08

I have a testcase I can give you to reproduce this bug.

Apologies for the delay in getting back to you, I was traveling. Little maven project. It throws up an unused import in SampleApp

mvn pmd:pmd

If you want I can open a new bug.

Last edit: thomaslennon 2013-04-08

bug254.zip

If you would like to refer to this comment somewhere else in this project, copy and paste the following link:

Andreas Dangel - 2013-04-08

Thanks for the test case.

I see now the problem, when it occurs: When declaring a RuntimeException via the @throws tag only in javadoc. As it is a runtime exception, the method itself might not declare this exception explicitly.

If you would like to refer to this comment somewhere else in this project, copy and paste the following link:

Andreas Dangel - 2013-04-08

status: not-reproducible --> accepted

milestone: PMD-5.0.3 --> PMD-5.0.x
If you would like to refer to this comment somewhere else in this project, copy and paste the following link:

Andreas Dangel - 2013-04-15

summary: False+ : UnusedImport with Javadoc @link --> False+ : UnusedImport with Javadoc @throws

status: accepted --> in-progress

Milestone: PMD-5.0.x --> PMD-5.0.4
If you would like to refer to this comment somewhere else in this project, copy and paste the following link:

Andreas Dangel - 2013-04-15

status: in-progress --> closed
If you would like to refer to this comment somewhere else in this project, copy and paste the following link:

Log in to post a comment.