Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.

Bug 325270

Summary:

[content assist] Parameter names are not displayed for static inner class of an external jar

Product:

[Eclipse Project] JDT

Reporter:

Satyam Kandula <satyam.kandula>

Component:

Core

Assignee:

Olivier Thomann <Olivier_Thomann>

Status:

VERIFIED FIXED

QA Contact:

Severity:

normal

Priority:

CC:

amj87.iitr, mauromol, Olivier_Thomann

Version:

3.6

Flags:

amj87.iitr: review+

Target Milestone:

3.7 M2

Hardware:

OS:

Windows XP

Whiteboard:

Attachments:

Description	Flags
Jar file to add in build path	none
Proposed fix	none
Proposed fix + regression test	none
New external library jar	none

Description Satyam Kandula

2010-09-14 11:19:23 EDT

Created attachment 178845 [details]
Jar file to add in build path

This is actually a FUP of bug 316937. 
bug 316937 does take care of inner classes of a jar, but doesn't work if the inner class is static! 

Use the attached jar file and try 
########
class Test {
 void test() {
    new pkg.Foo1.Bar1([CTRL-SPACE
 }
}
##########

Comment 1 Olivier Thomann

2010-09-14 11:29:12 EDT

I'll take a look.

Comment 2 Olivier Thomann

2010-09-14 11:51:57 EDT

Created attachment 178848 [details]
Proposed fix

I need to add regression test for this case.

Comment 3 Olivier Thomann

2010-09-14 12:22:25 EDT

Created attachment 178850 [details]
Proposed fix + regression test

Same fix with regression test.

Comment 4 Olivier Thomann

2010-09-14 12:23:30 EDT

Created attachment 178851 [details]
New external library jar

Jar file to put in /org.eclipse.jdt.core.tests.model/workspace/Completion

Comment 5 Ayushman Jain

2010-09-15 11:04:06 EDT

Patch looks good.

Comment 6 Olivier Thomann

2010-09-15 11:37:29 EDT

Released for 3.7M2.
Added regression test in:
org.eclipse.jdt.core.tests.model.CompletionTests#testCompletionMethodDeclaration17

Comment 7 Olivier Thomann

2010-09-21 17:57:29 EDT

Verified using I20100921-1024