Summary: | [1.5][assist] Camel case completion not working with statically imported methods | ||||||||||||
---|---|---|---|---|---|---|---|---|---|---|---|---|---|
Product: | [Eclipse Project] JDT | Reporter: | Mauro Molinari <mauromol> | ||||||||||
Component: | Core | Assignee: | Jay Arthanareeswaran <jarthana> | ||||||||||
Status: | VERIFIED FIXED | QA Contact: | |||||||||||
Severity: | normal | ||||||||||||
Priority: | P3 | CC: | david_audel | ||||||||||
Version: | 3.4 | ||||||||||||
Target Milestone: | 3.5 M7 | ||||||||||||
Hardware: | PC | ||||||||||||
OS: | Windows XP | ||||||||||||
Whiteboard: | |||||||||||||
Attachments: |
|
Description
Mauro Molinari
2008-09-10 04:36:28 EDT
Created attachment 132019 [details]
Proposed patch
Added the appropriate check before calling findLocalMethodsFromStaticImports method to ensure that camel case is considered. Added the test case in CompletionTests_1_5.testCamelCaseStaticMethodImport.
Created attachment 132049 [details]
Latest patch
Made corrections to the test case as suggested by David. Also excluded other changes that went along with the patch as part of a local merge.
Created attachment 132084 [details]
Latest patch
Fix made to handle the following case also. Test case modified too.
package a;
public class A {
public static void testMethodWithLongName(){}
public static void testMethodWithLongName2(){}
}
-----
package b;
import static a.A.testMethodWithLongName;
public class B {
public void b() {
tMWLN<= invoke code completion here
}
}
----
Created attachment 132185 [details]
Latest patch
Removed the code that was commented out earlier.
Comment on attachment 132185 [details]
Latest patch
This patch is good for me
Released for 3.5M7. Verified for 3.5M7 using I20090426-2000 |