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

Bug 261176

Summary: Changing projected method on a @noextend gets an API error
Product: [Eclipse Project] PDE Reporter: Francis Upton IV <francisu>
Component: API ToolsAssignee: Olivier Thomann <Olivier_Thomann>
Status: VERIFIED FIXED QA Contact:
Severity: normal    
Priority: P3 CC: Michael_Rennie, Olivier_Thomann
Version: 3.5   
Target Milestone: 3.5 M5   
Hardware: All   
OS: All   
Whiteboard:
Bug Depends on:    
Bug Blocks: 244697    

Description Francis Upton IV CLA 2009-01-15 04:39:30 EST 
Previously CommonNavigator had @noextend, and it had a projected method createCommonManager() that returned an internal class CommonNavigatorManager.

For 3.5, the @noextend was removed, and the CommonNavigatorManager class was moved to become API.

This should not result in an API error, but it does.



<?xml version="1.0" encoding="UTF-8" standalone="no"?>
<component id="org.eclipse.ui.navigator" version="2">
    <resource path="src/org/eclipse/ui/navigator/CommonNavigator.java" type="org.eclipse.ui.navigator.CommonNavigator">
        <filter id="338792546">
            <message_arguments>
                <message_argument value="org.eclipse.ui.navigator.CommonNavigator"/>
                <message_argument value="createCommonManager()"/>
            </message_arguments>
        </filter>
    </resource>
</component>
Comment 1 Olivier Thomann CLA 2009-01-15 13:46:11 EST 
Released for 3.5M5.
Regression tests added in org.eclipse.pde.api.tools.builder.tests.compatibility.ClassCompatibilityMethodTests#testRemoveProtectedAPIMethodNoExtend2I
org.eclipse.pde.api.tools.builder.tests.compatibility.ClassCompatibilityMethodTests#testRemoveProtectedAPIMethodNoExtend2F
org.eclipse.pde.api.tools.comparator.tests.MethodDeltaTests#test110

Michael, please verify.
Comment 2 Michael Rennie CLA 2009-03-11 10:52:01 EDT 
verified