Bug 197420 - [javadoc] No warning about unused method parameters if they're referenced by javadoc
Summary: [javadoc] No warning about unused method parameters if they're referenced by ...
Status: VERIFIED INVALID
Alias: None
Product: JDT
Classification: Eclipse Project
Component: Core (show other bugs)
Version: 3.3   Edit
Hardware: PC Linux
: P3 normal with 1 vote (vote)
Target Milestone: 3.4 M1   Edit
Assignee: Eric Jodet CLA
QA Contact:
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2007-07-22 10:52 EDT by Andrew ``Bass'' Shcheglov CLA
Modified: 2007-12-20 06:44 EST (History)
0 users

See Also:

Attachments
[screen shot] compilation options (60.70 KB, image/jpeg)
2007-07-24 06:46 EDT, Eric Jodet CLA 		no flags Details
View All Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Andrew ``Bass'' Shcheglov CLA 2007-07-22 10:52:21 EDT 
Build ID: I20070625-1500

Steps To Reproduce:
If an unused method parameter, say, p0, is referenced from within the method's javadoc comment (/** @param p0 */ is enough), Eclipse compiler produces no "The prarameter p0 is never read" warning.


More information:
I'd consider this a bug rather than a feature since such javadoc comments can be automatically generated by either Eclipse itself or any external facility, which, given the existing compiler behaviour, makes unused parameter hunting more difficult.

In case this is relevant: I'm using a GTK2 build on a Linux-IA32 box, JDK is 1.6.0_01, project source level is 1.6.
Comment 1 Frederic Fusier CLA 2007-07-22 11:18:44 EDT 
Similar issue than for bug 190970...
Comment 2 Eric Jodet CLA 2007-07-24 06:46:11 EDT 
Created attachment 74443 [details]
[screen shot] compilation options

Checking the default compilation options, the compiler is behaving as expected.
In this screen shot you'll notice that the default is to ignore unused parameters when they are documented with a @param tag.
If you disable this option (may be at project level), the a warning is raised as expected ("The parameter param is never read")
Comment 3 Eric Jodet CLA 2007-07-24 06:47:20 EDT 
(In reply to comment #2)
Closing as invalid
Comment 4 Andrew ``Bass'' Shcheglov CLA 2007-07-24 09:59:38 EDT 
Thanks guys.
I just was misguided since this option is on by default.
Comment 5 Frederic Fusier CLA 2007-12-20 06:44:06 EST 
Verified for 3.4M1