Bug 222900 - [Javadoc] Missing description is warned if valid description is on a new line
Summary: [Javadoc] Missing description is warned if valid description is on a new line
Status: VERIFIED FIXED
Alias: None
Product: JDT
Classification: Eclipse Project
Component: Core (show other bugs)
Version: 3.4   Edit
Hardware: All All
: P3 normal with 3 votes (vote)
Target Milestone: 3.5 M2   Edit
Assignee: Frederic Fusier CLA
QA Contact:
URL:
Whiteboard:
Keywords:
: 238590 244785 251160 252088 264415 (view as bug list)
Depends on: 73352
Blocks:
  Show dependency tree
 
Reported: 2008-03-16 14:46 EDT by Martin Oberhuber CLA
Modified: 2009-02-11 05:01 EST (History)
11 users (show)

See Also:


Attachments
Proposed patch (21.74 KB, patch)
2008-09-02 03:54 EDT, Frederic Fusier CLA
no flags Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Martin Oberhuber CLA 2008-03-16 14:46:53 EDT
Window > Preferences > Java > Code Style > Formatter
  Profile "Eclipse [built-in]"
  --> Commnets : New Line after @param tags is switched ON by default

This setting conflicts with the following new compiler warning feature if enabled, which was added in 3.4M3 as per bug 73352:

Window > Preferences > Java > Compiler > Javadoc
  Malformed Javadoc Comments: Warning
  Missing tag descriptions: Validate all tags

This is particularly problematic when "Format Code" is enabled as "On Save" operation, since the user has no chance getting it right in this case.

I think that either the default "Eclipse" profile should be changed to not add a new line after @param tags, or the "Missing tag descriptions" compiler warning should not issue a warning if the tag is there but on a new line.

Actually, the second solution seems better since the javadoc tool also accepts @param arguments on a new line, so the warning is incorrect in this case.
Comment 1 Martin Oberhuber CLA 2008-03-16 15:07:36 EDT
Changed summary, previous value was:
[formatter] Default "Eclipse" profile conflicts with "Javadoc tag missing argument" compiler warning for @param

The point is really that the "Javadoc: Description expected after this reference" warning is invalid if the reference is there but on a new line.
Comment 2 Philipe Mulet CLA 2008-03-16 23:49:31 EDT
Yes, I agree the warning should tolerate the new line.
Comment 3 Eric Jodet CLA 2008-06-26 10:18:35 EDT
*** Bug 238590 has been marked as a duplicate of this bug. ***
Comment 4 Mark Hobson CLA 2008-07-29 11:59:27 EDT
This bug causes valid source code to be littered with warnings when descriptions are on new lines - can this be targeted for 3.4.1 please?
Comment 5 Frederic Fusier CLA 2008-08-21 05:30:42 EDT
*** Bug 244785 has been marked as a duplicate of this bug. ***
Comment 6 Dani Megert CLA 2008-08-26 09:37:30 EDT
Can this be fixed for 3.5? If we enable to check all tags and not just @return we get many false warnings.
Comment 7 Mauro Molinari CLA 2008-08-26 12:21:22 EDT
(In reply to comment #6)
> Can this be fixed for 3.5? If we enable to check all tags and not just @return
> we get many false warnings.

I hope it is considered for 3.4.x, too... It's quite annoying...

Mauro.
Comment 8 Jerome Lanneluc CLA 2008-08-27 12:34:48 EDT
This is too late for 3.4.1. But we should have a look for 3.4.2 if it is a regression comparing to 3.3.2.
Comment 9 Kai Mechel CLA 2008-08-30 04:47:23 EDT
It would be great, if the fix could be scheduled for 3.4.2
As long as it is not fixed, I format the code without new lines after @param tags. But this doesn't look very pretty.
Comment 10 Frederic Fusier CLA 2008-09-02 03:54:12 EDT
Created attachment 111453 [details]
Proposed patch

Instead of verifying the end of line to report the missing description warning, the Javadoc parser waits either the next tag, the closing brace of the in-line tag or the end of the comment to warn if the description hasn't been set for the previous tag...
Comment 11 Frederic Fusier CLA 2008-09-02 03:58:49 EDT
I do not intend to backport this fix in 3.4 maintenance stream because
1) it's too big and risky
2) this cannot be considered as a regression as this problem only happens when user selects the new preference: "Missing tag description - Validate all standard tags".

The workaround for 3.4.x versions will be to keep using the default preference for this warning: "Missing tag description - Validate @return tag".
Comment 12 Frederic Fusier CLA 2008-09-02 04:09:29 EDT
Released for 3.5M2
Comment 13 David Audel CLA 2008-09-15 07:31:32 EDT
Verified for 3.5M2 using I20080914-2000
Comment 14 Frederic Fusier CLA 2008-10-17 03:57:38 EDT
*** Bug 251160 has been marked as a duplicate of this bug. ***
Comment 15 Frederic Fusier CLA 2008-10-25 11:49:23 EDT
*** Bug 252088 has been marked as a duplicate of this bug. ***
Comment 16 Frederic Fusier CLA 2008-10-25 11:50:35 EDT
*** Bug 252088 has been marked as a duplicate of this bug. ***
Comment 17 Frederic Fusier CLA 2009-02-11 05:01:19 EST
*** Bug 264415 has been marked as a duplicate of this bug. ***