Bugzilla – Full Text Bug Listing
| Summary: | [javadoc] incorrect warning about missing parameter javadoc when using many links | ||||||||||
|---|---|---|---|---|---|---|---|---|---|---|---|
| Product: | [Eclipse Project] JDT | Reporter: | Kevin Goeser <kevin> | ||||||||
| Component: | Core | Assignee: | Frederic Fusier <frederic_fusier> | ||||||||
| Status: | VERIFIED FIXED | QA Contact: | |||||||||
| Severity: | minor | ||||||||||
| Priority: | P3 | CC: | chris, david_audel, jerome_lanneluc | ||||||||
| Version: | 3.2.1 | ||||||||||
| Target Milestone: | 3.5 M2 | ||||||||||
| Hardware: | PC | ||||||||||
| OS: | Linux | ||||||||||
| Whiteboard: | |||||||||||
| Attachments: |
|
||||||||||
Created attachment 56963 [details]
sample java code
Sample code which creates incorrect warnings (first method).
Created attachment 101198 [details]
[patch] - proposed fix + test case (top v_866)
passes all jdt.core, ui and text tests
(thanks to the large number of param tags) The problem was due to an ArrayIndexOutOfBoundsException in the AbstractCommentParser which prevented the parser to proceed.
Problem fixed by resizing the array when needed.
Since this is not a regression comparing to 3.3.2, will target 3.5 Created attachment 112313 [details] Updated patch on top of v_912 The initial fix was good but the tests didn't fail when removing it => fix them + add a test for the test case provided in comment 1... Released for 3.5M2 Verified for 3.5M2 using I20080914-2000 |
Build ID: M20060921-0945 Steps To Reproduce: 1. Set the compile up to warn about missing @param / @return javadoc tags. 2. Create a function with many parameters. 3. Add @param tags for all parameters 4. Paste {@link SomeClass} to all parameter docs ... at some number of used @links, Eclipse / Java warns about missing @param tags. - Tested on Linux and Windows. - Might be a problem of Java, too (?) - Sun JDK 1.5.0.08 - Eclipse 3.2.1