Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.
Bug 337887 - [type hierarchy] MethodOverrideTester and Type Hierarchy don't consider flags etc.
Summary: [type hierarchy] MethodOverrideTester and Type Hierarchy don't consider flags...
Status: VERIFIED FIXED
Alias: None
Product: JDT
Classification: Eclipse Project
Component: UI (show other bugs)
Version: 3.7   Edit
Hardware: PC Windows 7
: P3 normal (vote)
Target Milestone: 3.7 M6   Edit
Assignee: Markus Keller CLA
QA Contact:
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2011-02-22 14:24 EST by Markus Keller CLA
Modified: 2011-03-08 13:23 EST (History)
1 user (show)

See Also:


Attachments
Fix (7.25 KB, patch)
2011-02-22 14:24 EST, Markus Keller 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-02-22 14:24:13 EST
Created attachment 189532 [details]
Fix

HEAD

MethodOverrideTester and the Type Hierarchy don't consider flags (private, static) and don't rule out constructors.

Example:

package javadoc;
public class Outer {

    /**
     * Outer ctor.
     */
    public Outer() {}
    /**
     * Outer method.
     */
    public String Outer() { return ""; }
    private void foo() { }
    public static void bar() { }
    
    class Inner {
        /**
         * Inner method.
         */
        public String Inner() { return ""; }
        /**
         * Inner ctor.
         */
        public Inner() {}
    }
}

class OuterSub extends Outer {
    /**
     * OuterSub method. {@inheritDoc}
     */
    @Override
    public String Outer() { return super.Outer(); }
    
    /**
     * foo method. {@inheritDoc}
     */
    public static void foo() { }
    public static void bar() { }
    
    class InnerSub extends Outer.Inner {
        /**
         * InnerSub method. {@inheritDoc}
         */
        @Override
        public String Inner() { return super.Inner(); }
    }
}
Comment 1 Markus Keller CLA 2011-02-22 14:29:29 EST
Fixed in HEAD.
Comment 2 Rajesh CLA 2011-03-08 13:23:02 EST
Verified with I20110307-2110