Bug 170889 - [5.0][content assist] strange label for 'class' proposal
Summary: [5.0][content assist] strange label for 'class' proposal
Status: VERIFIED FIXED
Alias: None
Product: JDT
Classification: Eclipse Project
Component: Core (show other bugs)
Version: 3.3   Edit
Hardware: Macintosh Mac OS X - Carbon (unsup.)
: P3 normal (vote)
Target Milestone: 3.4 M1   Edit
Assignee: David Audel CLA
QA Contact:
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2007-01-18 06:11 EST by Andre Weinand CLA
Modified: 2007-08-03 08:19 EDT (History)
1 user (show)

See Also:

Attachments
content assist (15.66 KB, image/png)
2007-01-18 13:30 EST, Andre Weinand CLA 		no flags Details
Proposed fix (2.44 KB, patch)
2007-06-29 06:10 EDT, David Audel CLA 		no flags Details | Diff
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 Andre Weinand CLA 2007-01-18 06:11:38 EST 
Eclipse 3.3 M3
JRE 1.5

- open content assist for a Java class that is not available in the JDK, e.g. position the caret after the dot in "Foo." and press control-space.
Observe: one proposal is "class  Class<org/weinand/Foo>"
I don't understand why the fully qualified type is shown as a directory path and not in Java "."-syntax.
Comment 1 Dani Megert CLA 2007-01-18 12:35:55 EST 
Can you paste a code snippet here. I have trouble parsing your comment ;-)
Comment 2 Andre Weinand CLA 2007-01-18 13:30:36 EST 
Created attachment 57095 [details]
content assist
Comment 3 Andre Weinand CLA 2007-01-18 13:33:11 EST 
Sorry, I forgot to attach a screenshot...
Comment 4 Dani Megert CLA 2007-01-19 07:05:20 EST 
The proposal we get from JDT Core looks wrong:

[FIELD_REF]{completion:class, declSign:, sign:Ljava.lang.Class<Ljava/lang/String;>;, declKey:, key:, name:class, [70,70], relevance=20}

Test Case:
public class C {
  void foo() {
    String.<code assist>
  }
}
Comment 5 David Audel CLA 2007-06-29 06:10:35 EDT 
Created attachment 72763 [details]
Proposed fix
Comment 6 David Audel CLA 2007-06-29 06:13:06 EDT 
Released for 3.4M1.

Test updated
  CompletionTests_1_5#test0228() -> test0229()
Comment 7 Frederic Fusier CLA 2007-08-03 08:19:48 EDT 
Verified for 3.4M1 using build I20070802-0800.