Bug 97316 - [5.0][typing] Problems with quick fix "Add javadoc comment"
Summary: [5.0][typing] Problems with quick fix "Add javadoc comment"
Status: VERIFIED FIXED
Alias: None
Product: JDT
Classification: Eclipse Project
Component: UI (show other bugs)
Version: 3.1   Edit
Hardware: PC Windows XP
: P3 normal (vote)
Target Milestone: 3.2 M3   Edit
Assignee: Martin Aeschlimann CLA
QA Contact:
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2005-05-30 13:11 EDT by Andrey Loskutov CLA
Modified: 2005-11-01 10:07 EST (History)
1 user (show)

See Also:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Andrey Loskutov CLA 2005-05-30 13:11:34 EDT
Consider this code:

public class A implements Comparable<String> {
    public int compareTo(String s) {
        return 0;
    }
}

This comment is generated actually by quick fix "Add javadoc comment" for 
compareTo() method:

public class A implements Comparable<String> {

    /* (non-Javadoc)
     * @param s
     * @return
     * @see java.lang.Comparable#compareTo(T)
     */
    public int compareTo(String s) {
        return 0;
    }
}

The "(T)" in java.lang.Comparable#compareTo(T) should be "(Object)", otherwise 
if I change the (non-Javadoc) comment to the javadoc comment, a warning:  
"Javadoc: T cannot be resolved to a type" appear.

See also probably related (not - quick-fix) discussion in the bug 83127.
Comment 1 Dani Megert CLA 2005-05-30 13:21:30 EDT

*** This bug has been marked as a duplicate of 87868 ***
Comment 2 Frederic Fusier CLA 2005-09-16 06:26:38 EDT
Dani,
Why did you put a quick-fix issue duplicate as code assist bug 87868?
As now Javadoc code assist will be performed by JDT/Core, I need to know
precisely the relationship between these two issues
Thanks
Comment 3 Dani Megert CLA 2005-09-16 07:16:09 EDT
Accidentally marked as dup.
Comment 4 Martin Aeschlimann CLA 2005-09-28 13:00:28 EDT
fixed > 20050928
Comment 5 Philip Mayer CLA 2005-11-01 08:55:50 EST
Verified in I20051031-2000.
Comment 6 Dirk Baeumer CLA 2005-11-01 10:07:31 EST
Marking as verified...