Bug 326514 - Most Javadocs in JavaCore.java are missing </p>.
Summary: Most Javadocs in JavaCore.java are missing </p>.
Status: VERIFIED FIXED
Alias: None
Product: JDT
Classification: Eclipse Project
Component: Core (show other bugs)
Version: 3.7   Edit
Hardware: All All
: P3 trivial (vote)
Target Milestone: 4.3 M1   Edit
Assignee: Tomasz Zarna CLA
QA Contact:
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2010-09-29 07:11 EDT by Dani Megert CLA
Modified: 2012-08-07 05:05 EDT (History)
3 users (show)

See Also:


Attachments
Fix (123.96 KB, patch)
2012-07-18 05:49 EDT, Tomasz Zarna CLA
no flags Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Dani Megert CLA 2010-09-29 07:11:20 EDT
HEAD.

Most Javadocs in JavaCore.java are missing </p>.
Comment 1 Ayushman Jain CLA 2010-09-29 07:42:17 EDT
I'll take care of this after releasing fixes for bug 325342 and bug 322581
Comment 2 Tomasz Zarna CLA 2012-07-13 09:57:50 EDT
Ayush, I will take this one over. It's a perfect way of getting familiar with Java Core API and doing something useful at the same time.
Comment 3 Ayushman Jain CLA 2012-07-13 12:25:39 EDT
Sure, thanks Tom! Also remember to validate the markup when you're done so that it passes the CHKPII tests
Comment 4 Srikanth Sankaran CLA 2012-07-16 09:02:46 EDT
http://validator.w3.org/ is a possibility.
Comment 5 Tomasz Zarna CLA 2012-07-18 05:49:25 EDT
Created attachment 218860 [details]
Fix

The number of opening <p> and closing </p> attrs equals now ie it's 339. The w3 validator shows no errors for the generated javadoc.
Comment 6 Ayushman Jain CLA 2012-07-18 05:51:16 EDT
(In reply to comment #5)
> Created attachment 218860 [details]
> Fix
> 
> The number of opening <p> and closing </p> attrs equals now ie it's 339. The w3
> validator shows no errors for the generated javadoc.
Thanks Tom, I'll release this fix for 4.3M1
Comment 7 Tomasz Zarna CLA 2012-07-18 13:00:11 EDT
(In reply to comment #5)
> Created attachment 218860 [details]
> Fix

The fix has been push to Gerrit as https://git.eclipse.org/r/#/c/6846
Comment 8 Ayushman Jain CLA 2012-07-20 05:05:43 EDT
Reviewed successfully through Gerrit and merged to master via http://git.eclipse.org/c/jdt/eclipse.jdt.core.git/commit/?id=166bb18dfc5a3ebc7bd8aa135b74b0c874fa431d
w00t!
Comment 9 Jay Arthanareeswaran CLA 2012-08-07 05:05:06 EDT
Verified for 4.3 M1 by code inspection.