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

Bug 324076

Summary:

Removing @noextend and adding @nooverride results API tool problem being reported

Product:

[Eclipse Project] PDE

Reporter:

Dani Megert <daniel_megert>

Component:

API Tools

Assignee:

Olivier Thomann <Olivier_Thomann>

Status:

VERIFIED FIXED

QA Contact:

Severity:

normal

Priority:

CC:

darin.eclipse, Michael_Rennie, Olivier_Thomann, remy.suen

Version:

3.6

Flags:

Michael_Rennie: review+
Michael_Rennie: review? (darin.eclipse)

Target Milestone:

3.7 M3

Hardware:

OS:

Windows XP

Whiteboard:

Attachments:

Description	Flags
Proposed fix + regression test	none

Description Dani Megert

2010-08-31 07:19:02 EDT

3.6.

Removing @noextend on a class and adding @nooverride on its methods results API tool error being reported.

Comment 1 Dani Megert

2010-08-31 08:30:49 EDT

>3.6.
N20100829-2000 but probably also in 3.6.

For steps see bug 324079 comment 0.

Comment 2 Darin Wright

2010-08-31 10:56:08 EDT

I see... Since the class was @noextend, you were not previously allowed to override the method... so the @nooverride restriction should not be considered as new.

Comment 3 Olivier Thomann

2010-09-09 13:48:12 EDT

Created attachment 178540 [details]
Proposed fix + regression test

Comment 4 Olivier Thomann

2010-09-09 13:48:27 EDT

Darin, Michael, please review.

Comment 5 Michael Rennie

2010-09-27 11:31:52 EDT

+1 works for me and all tests pass. Adding Darin as a reviewer like Olivier intended.

Comment 6 Olivier Thomann

2010-10-06 14:11:06 EDT

Released for 3.7M3.

Comment 7 Dani Megert

2010-10-11 10:24:27 EDT

Verified in N20101010-2000.