Attachment #188689 for bug #336821




					if (scope.enclosingSourceType().isCompatibleWith(fieldRef.actualReceiverType)) {
						fieldRef.bits |= ASTNode.SuperAccess;
					}
					ReferenceBinding resolvedType = (ReferenceBinding) fieldRef.actualReceiverType;
					char[] selector = fieldRef.token;
					boolean isConstructor = false;
					if (CharOperation.equals(resolvedType.sourceName(), fieldRef.token)) {
						selector = TypeConstants.INIT;
						isConstructor = true;
					}
					if (isConstructor) {
						fieldRef.methodBinding = scope.getConstructor(resolvedType, Binding.NO_TYPES, fieldRef);
					} else {
						fieldRef.methodBinding = scope.findMethod(resolvedType, selector, Binding.NO_TYPES, fieldRef);
					}
				}
			}





/*******************************************************************************
 * Copyright (c) 2000, 2011 IBM Corporation and others.
 * All rights reserved. This program and the accompanying materials
 * are made available under the terms of the Eclipse Public License v1.0
 * which accompanies this distribution, and is available at

 *******************************************************************************/
package org.eclipse.jdt.internal.compiler.ast;

import org.eclipse.jdt.core.compiler.CharOperation;
import org.eclipse.jdt.internal.compiler.ASTVisitor;
import org.eclipse.jdt.internal.compiler.impl.Constant;
import org.eclipse.jdt.internal.compiler.lookup.*;

			}
			if (this.actualReceiverType instanceof ReferenceBinding) {
				ReferenceBinding refBinding = (ReferenceBinding) this.actualReceiverType;
				char[] selector = this.token;
				MethodBinding possibleMethod = null;
				boolean isConstructor = false;
				if (CharOperation.equals(this.actualReceiverType.sourceName(), selector)) {
					selector = TypeConstants.INIT;
					isConstructor = true;
				}
				if (isConstructor) {
					possibleMethod = scope.getConstructor(refBinding, Binding.NO_TYPES, this);
				} else {
					possibleMethod = this.receiver.isThis()
						? scope.getImplicitMethod(selector, Binding.NO_TYPES, this)
						: scope.getMethod(refBinding, selector, Binding.NO_TYPES, this);
				}
				if (possibleMethod.isValidBinding()) {
					this.methodBinding = possibleMethod;
				} else {

Lines 1-5 Link Here

(-)src/org/eclipse/jdt/core/tests/dom/ASTConverterJavadocTest.java (-1 / +6 lines)
1	/*******************************************************************************	1	/*******************************************************************************
2	* Copyright (c) 2000, 2010 IBM Corporation and others.	2	* Copyright (c) 2000, 2011 IBM Corporation and others.
3	* All rights reserved. This program and the accompanying materials	3	* All rights reserved. This program and the accompanying materials
4	* are made available under the terms of the Eclipse Public License v1.0	4	* are made available under the terms of the Eclipse Public License v1.0
5	* which accompanies this distribution, and is available at	5	* which accompanies this distribution, and is available at
Lines 3296-3299 Link Here
3296	public void test109() throws JavaModelException {	3296	public void test109() throws JavaModelException {
3297	verifyComments("test109");	3297	verifyComments("test109");
3298	}	3298	}
		3299	// https://bugs.eclipse.org/bugs/show_bug.cgi?id=336821
		3300	public void testBug336821() throws JavaModelException {
		3301	ICompilationUnit unit = getCompilationUnit("Converter" , "src", "javadoc.testBug336821", "Try.java"); //$NON-NLS-1$ //$NON-NLS-2$ //$NON-NLS-3$ //$NON-NLS-4$
		3302	verifyComments(unit);
		3303	}
3299	}	3304	}




package javadoc.testBug336821;
/**
 * First ref should resolve to constructor
 * @see #Try
 * @see #Try(int, String)
 * @see #foo
 */
public class Try {
    public Try(int i, String message) {
        System.out.println(message + i);
    }

    public void foo(int i, String message) {
        System.out.println(message + i);
    }
}

Return to bug 336821

Lines 399-405 Link Here

(-)compiler/org/eclipse/jdt/internal/compiler/ast/Javadoc.java (-1 / +12 lines)
399	if (scope.enclosingSourceType().isCompatibleWith(fieldRef.actualReceiverType)) {	399	if (scope.enclosingSourceType().isCompatibleWith(fieldRef.actualReceiverType)) {
400	fieldRef.bits \|= ASTNode.SuperAccess;	400	fieldRef.bits \|= ASTNode.SuperAccess;
401	}	401	}
402	fieldRef.methodBinding = scope.findMethod((ReferenceBinding)fieldRef.actualReceiverType, fieldRef.token, new TypeBinding[0], fieldRef);	402	ReferenceBinding resolvedType = (ReferenceBinding) fieldRef.actualReceiverType;
		403	char[] selector = fieldRef.token;
		404	boolean isConstructor = false;
		405	if (CharOperation.equals(resolvedType.sourceName(), fieldRef.token)) {
		406	selector = TypeConstants.INIT;
		407	isConstructor = true;
		408	}
		409	if (isConstructor) {
		410	fieldRef.methodBinding = scope.getConstructor(resolvedType, Binding.NO_TYPES, fieldRef);
		411	} else {
		412	fieldRef.methodBinding = scope.findMethod(resolvedType, selector, Binding.NO_TYPES, fieldRef);
		413	}
403	}	414	}
404	}	415	}
405		416

Added Link Here

(-)workspace/Converter/src/javadoc/testBug336821/Try.java (+16 lines)
1	package javadoc.testBug336821;
2	/**
3	* First ref should resolve to constructor
4	* @see #Try
5	* @see #Try(int, String)
6	* @see #foo
7	*/
8	public class Try {
9	public Try(int i, String message) {
10	System.out.println(message + i);
11	}
12
13	public void foo(int i, String message) {
14	System.out.println(message + i);
15	}
16	}