Bug 50838 - Javadoc bindings: No bindings in constructor ref parameter
Summary: Javadoc bindings: No bindings in constructor ref parameter
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 M7   Edit
Assignee: Frederic Fusier CLA
QA Contact:
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2004-01-29 06:05 EST by Martin Aeschlimann CLA
Modified: 2004-02-11 10:51 EST (History)
0 users

See Also:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Martin Aeschlimann CLA 2004-01-29 06:05:35 EST
20040129

In the following code, 'Map' in the Javadoc comment has no binding.
Also HashMap in '#HashMap' has no binding

package x;

import java.util.HashMap;
import java.util.Map;
import java.util.Vector;


public class AA {
	
	/**
	 * @see HashMap#HashMap(Map)
	 */
	public void foo(Vector v, Vector vd) {
		Map map;
	}
}
Comment 1 Martin Aeschlimann CLA 2004-01-29 06:24:56 EST
It has not only to do with constructors:

In this code 'Map' also has no binding.
('putAll' has it)

package x;

import java.util.HashMap;
import java.util.Map;
import java.util.Vector;


public class AA {
	
	/**
	 * @see HashMap#putAll(Map)
	 */
	public void foo(Vector v, Vector vd) {
	}
}
Comment 2 Frederic Fusier CLA 2004-01-29 11:32:07 EST
Fixed.

Parameters were forgot during conversion (and also during tests)...
Changes made in:
 - ASTConverter.convert(Javadoc),
 - DefaultBindingResolver.resolveName(Name) to take into account that name
   can refers to an AllocationExpression,
 - JavadocArgumentExpression to store binding in its argument as this is this
   one which is used while resolving binding.
 - compiler.ast.Javadoc.getNodeStartingAt(int) to look starting positions for
   arguments
 - JavadocParser.createArgumentReference(...) as starting position was wrongly
   computed

Test cases added in jdt.core.tests.dom.ASTConverterJavadocTest
Comment 3 David Audel CLA 2004-02-11 10:51:56 EST
Verified for 3.0M7