Bug 95219 - [Ctrl+Click] is enabled on everything except for keywords
Summary: [Ctrl+Click] is enabled on everything except for keywords
Status: RESOLVED FIXED
Alias: None
Product: CDT
Classification: Tools
Component: cdt-core (show other bugs)
Version: 3.0   Edit
Hardware: PC Windows XP
: P3 normal (vote)
Target Milestone: 3.0   Edit
Assignee: Devin Steffler CLA
QA Contact:
URL:
Whiteboard:
Keywords: contributed
Depends on:
Blocks:
 
Reported: 2005-05-13 15:48 EDT by Devin Steffler CLA
Modified: 2009-01-09 16:57 EST (History)
0 users

See Also:


Attachments
cdt_ui patch (2.33 KB, patch)
2005-05-16 15:35 EDT, Devin Steffler CLA
bjorn.freeman-benson: iplog+
Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Devin Steffler CLA 2005-05-13 15:48:43 EDT
Ctrl+Click is enabled on all characters except for those that belong to a 
keyword.  It should only be available on IASTNames I would think...

Example, hold down Ctrl and move the mouse over ';' or some comments, or some 
spaces, etc...

// take the example from C++ spec 14.3.2-5:
template <class T> struct A {
	void f(int);
	template <class T2> void f(T2);
	};
	template <> void A<int>::f(int) { } // nontemplate member
	template <> template <> void A<int>::f<>(int) { } // template member
	int main()
	{
	A<char> ac;
	ac.f(1); //nontemplate
	ac.f('c'); //template
	ac.f<>(1); //template
	}
Comment 1 Devin Steffler CLA 2005-05-16 15:35:32 EDT
Created attachment 21222 [details]
cdt_ui patch
Comment 2 Devin Steffler CLA 2005-05-16 16:02:44 EDT
Note that this patch only fixes the parts that are not java identifier parts 
and numbers.

Here's what used to be underlined but now isn't with this patch (with this 
example):
(int)
{
}
<
>
.
(
)
'
1
/
 <-- space :)
:
;

Although the words that are commented are still being underlined with 
Ctrl+Mouse hover.  There is a TODO that mentions JDT does a code complete on 
the text to see if it should be highlighted or not.  I think that's too 
expensive for us.
Comment 3 Devin Steffler CLA 2005-05-17 09:00:59 EDT
Patch applied to HEAD.