Bug 103304 - [Javadoc] Wrong reference proposal for inner classes.
Summary: [Javadoc] Wrong reference proposal for inner classes.
Status: VERIFIED FIXED
Alias: None
Product: JDT
Classification: Eclipse Project
Component: Core (show other bugs)
Version: 3.1   Edit
Hardware: PC Windows XP
: P3 normal (vote)
Target Milestone: 3.3 M1   Edit
Assignee: Frederic Fusier CLA
QA Contact:
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2005-07-11 07:23 EDT by Eric Bodden CLA
Modified: 2006-08-04 08:26 EDT (History)
1 user (show)

See Also:

Attachments
the test case (7.88 KB, text/plain)
2005-07-11 07:24 EDT, Eric Bodden 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 Eric Bodden CLA 2005-07-11 07:23:42 EDT 
The attached file hold an interface IAFAState with an inner class 
ValidationException.

The javadoc comment in line 117 (interface scope) references the constructor of 
this class:

     * @see ValidationException#IAFAState.ValidationException(String, IAFAState)

The problem is: Eclipse issues a warning if the reference is written like that.
It suggest instead:

     * @see ValidationException#ValidationException(String, IAFAState)

However, Javadoc needs the former line.
Comment 1 Eric Bodden CLA 2005-07-11 07:24:20 EDT 
Created attachment 24525 [details]
the test case
Comment 2 Frederic Fusier CLA 2006-04-12 13:58:19 EDT 

*** This bug has been marked as a duplicate of 96237 ***
Comment 3 Frederic Fusier CLA 2006-06-27 10:28:16 EDT 
Finally this is not a duplicate of bug 92367. This is a javadoc tool bug (http://bugs.sun.com/bugdatabase/view_bug.do?bug_id=4288720) and we need to decide whether we accept both syntaxes or reject one of them...?
Comment 4 Frederic Fusier CLA 2006-07-02 08:34:26 EDT 
The best solution I can see for now is to accept both syntaxes. I doubt that this problem will be fixed as it exists for a long time now. I'll keep on eye on it in case Sun decides to finally fix it. Then, we'll open a new bug to reject the incorrect syntax...

Fixed and released for 3.3 M1 in HEAD.
Comment 5 Maxime Daniel CLA 2006-08-04 08:26:10 EDT 
Verified for 3.3 M1 using build I20060804-0010.