Bug 68025 - Javadoc processing does not detect some wrong links
Summary: Javadoc processing does not detect some wrong links
Status: RESOLVED 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.1   Edit
Assignee: Frederic Fusier CLA
QA Contact:
URL:
Whiteboard:
Keywords:
Depends on:
Blocks: 69275
  Show dependency tree
 
Reported: 2004-06-21 11:52 EDT by Dani Megert CLA
Modified: 2004-09-06 10:26 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 Dani Megert CLA 2004-06-21 11:52:27 EDT 
3.0 RC3

have the following Javadoc:
/**
 * @see IJavaElement#getElementName#bug
 * or closer to the reality (COPY is a constant):
 * @see org.eclipse.ui.actions.ActionFactory#COPY#getId
 */

this should report that #bug and #getId are not correct. We have several
warnings in the SDK's Javadoc logs due to this.
Comment 1 Frederic Fusier CLA 2004-06-21 12:51:02 EDT 
You're right, compiler should complain about these invalid syntax.
Javadoc tool raises 2 warnings here:
warning - Tag @see: reference not found:
org.eclipse.jdt.core.IJavaElement#getElementName#bug
 or closer to the reality (COPY is a constant):
warning - Tag @see: reference not found:
org.eclipse.ui.actions.ActionFactory#COPY#getId
Comment 2 Frederic Fusier CLA 2004-06-21 12:51:34 EDT 
deferred post 3.0
Comment 3 Frederic Fusier CLA 2004-07-05 09:01:20 EDT 
Candidate for 3.0.1
Comment 4 Frederic Fusier CLA 2004-07-08 09:41:20 EDT 
Fixed.

In this case, compiler now reports following warning:
"Javadoc: Malformed reference (missing end space separator)"

[jdt-core-internal]
Change done in AbstractCommentParser methods:
- parseArgument(Object)
- parseMember(Object)
- parseReference(boolean)
- parseQualifiedName(boolean)
also add methods:
- verifyEndLine(int)
- verifyCharsAfterName()
Test cases added in JavadocTestMixed and ASTConverterJavadocTest.
Comment 5 Frederic Fusier CLA 2004-07-08 09:41:32 EDT 
Released in HEAD.
Comment 6 David Audel CLA 2004-09-06 10:26:25 EDT 
Verified for 3.0.1 RC1