Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.
Bug 322129 - "Unknown Hyperlink" if more than one hyperlink available
Summary: "Unknown Hyperlink" if more than one hyperlink available
Status: RESOLVED FIXED
Alias: None
Product: CDT
Classification: Tools
Component: cdt-editor (show other bugs)
Version: 6.0.2   Edit
Hardware: PC Windows XP
: P3 normal (vote)
Target Milestone: 7.0.1   Edit
Assignee: Anton Leherbauer CLA
QA Contact: Anton Leherbauer CLA
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2010-08-09 08:48 EDT by Marko Tomljenovic CLA
Modified: 2010-11-15 07:23 EST (History)
0 users

See Also:

Attachments
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 Marko Tomljenovic CLA 2010-08-09 08:48:41 EDT 
Build Identifier: CDT 6.0.2

We have implemented a hyper link provider so that more than hyperlink is available when pressing Ctrl and hovering over a c element.

Now the hyperlink provided by CDT is always showing "Unknown Hyperlink".
The reason for that is that the class "CElementHyperlink" always returns null as the hyperlink text. It would be better to returns the text/description/tooltip of the corresponding action.

Reproducible: Always

Steps to Reproduce:
See details.
Comment 1 Anton Leherbauer CLA 2010-08-13 08:04:11 EDT 
Fixed in HEAD and cdt_7_0.
Comment 2 CDT Genie CLA 2010-08-13 08:23:04 EDT 
*** cdt cvs genie on behalf of aleherbau ***
Bug 322129 - "Unknown Hyperlink" if more than hyperlink available

[*] CElementHyperlink.java 1.5 http://dev.eclipse.org/viewcvs/index.cgi/org.eclipse.cdt-core/org.eclipse.cdt.ui/src/org/eclipse/cdt/internal/ui/editor/CElementHyperlink.java?root=Tools_Project&r1=1.4&r2=1.5
Comment 3 Marko Tomljenovic CLA 2010-11-15 03:25:50 EST 
The link now has a better name.
There is only a minor glitch still available. The 'Goto Declaration' command now is shown as '&Goto Declaration'. The & sign should be removed from the text
Comment 4 Anton Leherbauer CLA 2010-11-15 06:48:43 EST 
Thanks, I checked in a fix to remove the mnemonics from the hyperlink text.