Bug 220171 - [assist] No suggesting 'hasGenericError' variable
Summary: [assist] No suggesting 'hasGenericError' variable
Status: VERIFIED FIXED
Alias: None
Product: JDT
Classification: Eclipse Project
Component: Core (show other bugs)
Version: 3.4   Edit
Hardware: PC Windows XP
: P3 normal (vote)
Target Milestone: 3.4 M6   Edit
Assignee: David Audel CLA
QA Contact:
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2008-02-25 06:16 EST by Philipe Mulet CLA
Modified: 2008-03-25 11:19 EDT (History)
2 users (show)

See Also:

Attachments
source of the unit being edited (11.14 KB, text/plain)
2008-02-25 06:16 EST, Philipe Mulet CLA 		no flags Details
Proposed fix (4.12 KB, patch)
2008-02-26 05:59 EST, 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 Philipe Mulet CLA 2008-02-25 06:16:11 EST 
Created attachment 90624 [details]
source of the unit being edited

Build id: I20080207-1530

Was editing ParameterizedSingleTypeReference.java, and having attached contents. Tried completing on local variable reference at line 223:

		if (checkBounds && !hasGen<CODEASSIST HERE>

and completion did not yield 'hasGenericError' as expected.

Also note that it proposed still some type completions, and performed a side effect on prefix in editor (turned the lowercase 'h' into 'H') !?
Comment 1 Philipe Mulet CLA 2008-02-25 06:17:17 EST 
Dani - is text transforming the 'h' into 'H' ?
Comment 2 Philipe Mulet CLA 2008-02-25 06:38:48 EST 
To clarify, the extra 'h'->'H' transformation occurs at the moment I pressed ctrl-space, when the popup opened with proposals (type names starting with capital H letters). But I did not select any proposal for insertion, and option for auto-inserting/auto-activation are disabled.
Comment 3 Philipe Mulet CLA 2008-02-25 06:47:29 EST 
Filed bug 220176 for 'h'->'H' conversion (likely a text bug)
Comment 4 David Audel CLA 2008-02-26 05:59:09 EST 
Created attachment 90736 [details]
Proposed fix
Comment 5 David Audel CLA 2008-02-26 06:02:02 EST 
Released for 3.4M6.

Test added
  CompletionTests#testNameWithUnresolvedReferences018()

The bug was in the algorithm which search unresolved reference.
Comment 6 Jerome Lanneluc CLA 2008-03-25 11:19:12 EDT 
Verified for 3.4M6 using I20080325-0100