Bug 66573 - Shouldn't bind to local constructs
Summary: Shouldn't bind to local constructs
Status: VERIFIED FIXED
Alias: None
Product: JDT
Classification: Eclipse Project
Component: Core (show other bugs)
Version: 3.0   Edit
Hardware: PC Windows XP
: P3 normal (vote)
Target Milestone: 3.0 RC2   Edit
Assignee: Frederic Fusier CLA
QA Contact:
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2004-06-10 15:28 EDT by Philipe Mulet CLA
Modified: 2004-06-11 09:51 EDT (History)
0 users

See Also:

Attachments
Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Philipe Mulet CLA 2004-06-10 15:28:21 EDT 
Build 3.0RC1

Following code should flag malformed @see tag:

public class X {
    /**
     * @see Local
     */
    void foo() {
        class Local { 
            // shouldn't be seen from javadoc
         }
    }
}
Comment 1 Philipe Mulet CLA 2004-06-10 15:30:24 EDT 
In AbstractMethodDeclaration#resolve(ClassScope)
the resolution order is wrong. Therefore javadoc resolution would see 
constructs defined in method scope, like local types here.

Suggest to reorder as follow:
			bindArguments(); 
			resolveJavadoc();
			bindThrownExceptions();
			resolveStatements();
Comment 2 Philipe Mulet CLA 2004-06-10 17:31:36 EDT 
Should rather be:
			bindArguments(); 
			bindThrownExceptions();
			resolveJavadoc();
			resolveStatements();
Comment 3 Philipe Mulet CLA 2004-06-10 17:57:45 EDT 
Released for integration. Frederic, pls add regression tests.
Comment 4 Frederic Fusier CLA 2004-06-10 18:34:23 EDT 
Test case added in JavadocTestMixed
Comment 5 David Audel CLA 2004-06-11 09:51:59 EDT 
Verified for 3.0RC2 I20040611