Summary: | Code assist from Javadoc | ||||||
---|---|---|---|---|---|---|---|
Product: | [Eclipse Project] JDT | Reporter: | prabhu <prabhu_p> | ||||
Component: | Core | Assignee: | David Audel <david_audel> | ||||
Status: | VERIFIED FIXED | QA Contact: | |||||
Severity: | enhancement | ||||||
Priority: | P3 | CC: | daniel_megert, eclipse | ||||
Version: | 3.1 | ||||||
Target Milestone: | 3.2 M4 | ||||||
Hardware: | PC | ||||||
OS: | Windows XP | ||||||
Whiteboard: | |||||||
Bug Depends on: | 84750, 110173 | ||||||
Bug Blocks: | |||||||
Attachments: |
|
Description
prabhu
2005-06-02 14:09:49 EDT
I believe you talk about the additional info popup at the side of the code assist window, where you would like to see javadoc information. Then this is a dup of 41421. *** This bug has been marked as a duplicate of 41421 *** JDT Text might in the future extract the Javadoc for the hovers from the Javadoc files (though also that could be regarded as a JDT Core job) however, code assist is definitely a JDT Core feature and hence they have to comment about this PR. Ok. The only thing I could think that core could retrieve from javadoc (instead of source) is the parameter names of methods. As says by tom (comment 3) the only thing that jdtcore could need from javadoc is parameter names of methods. Perhaps in the future, jdtcore CompletionProposal could give the javadoc of the proposed element. Now that we have fixed 110173, this is doable. I don't think there is anything to do here once the method getParameterNames() returns the names from the attached javadoc. We simply need to ensure that this method is called even if no source is attached. This is about the only change I see. Created attachment 29720 [details]
Proposed fix
With this change, the getParameterNames() method is called for a binary method
even if the no source is attached. This allows the method to return names from
the attached javadoc if any.
David, please review patch and apply. The patch is good. Fixed and test added CompletionTests#testParameterNames1() Verified for 3.2 M4 using build I20051212-0010 |