Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.
Bug 350496 - [1.7] @PolymorphicSignature IMethods are missing the annotation
Summary: [1.7] @PolymorphicSignature IMethods are missing the annotation
Status: VERIFIED FIXED
Alias: None
Product: JDT
Classification: Eclipse Project
Component: Core (show other bugs)
Version: 3.7   Edit
Hardware: PC Windows 7
: P3 normal (vote)
Target Milestone: 3.7.1   Edit
Assignee: Olivier Thomann CLA
QA Contact:
URL:
Whiteboard:
Keywords:
Depends on:
Blocks: 350087
  Show dependency tree
 
Reported: 2011-06-27 15:57 EDT by Markus Keller CLA
Modified: 2011-08-05 02:54 EDT (History)
2 users (show)

See Also:


Attachments
Proposed fix + regression test (7.16 KB, patch)
2011-06-28 10:58 EDT, Olivier Thomann CLA
no flags Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Markus Keller CLA 2011-06-27 15:57:08 EDT
BETA_JAVA7

The @PolymorphicSignature methods are missing their annotation. IMethod#getAnnotations() returns an empty array, although the annotation has @Retention(RUNTIME).
Comment 1 Olivier Thomann CLA 2011-06-27 18:19:00 EDT
I guess you are talking about the declaration of the methods.
Comment 2 Markus Keller CLA 2011-06-28 10:24:48 EDT
> I guess you are talking about the declaration of the methods.

Regarding getAnnotations(), there's no difference between the declaration element (may be unresolved) and a reference (with isResolved() == true).
Comment 3 Olivier Thomann CLA 2011-06-28 10:36:42 EDT
This annotation is seen as a standard annotation like Deprecated or Documented,...
If you want it to be returned as part of the annotations collection, it should not be part of the standard annotations.
Comment 4 Olivier Thomann CLA 2011-06-28 10:40:55 EDT
I forgot to add this standard annotation back into the annotation array. Should be fixed soon.
Comment 5 Olivier Thomann CLA 2011-06-28 10:58:23 EDT
Created attachment 198742 [details]
Proposed fix + regression test
Comment 6 Olivier Thomann CLA 2011-06-28 11:04:44 EDT
Released in BETA_JAVA7 branch.
Comment 7 Raksha Vasisht CLA 2011-07-20 06:10:16 EDT
Verified.