Bug 28098 - Code Formatter doesn't format JavaDoc indentation
Summary: Code Formatter doesn't format JavaDoc indentation
Status: VERIFIED FIXED
Alias: None
Product: JDT
Classification: Eclipse Project
Component: Core (show other bugs)
Version: 2.1   Edit
Hardware: PC Windows NT
: P3 normal (vote)
Target Milestone: 3.0 M4   Edit
Assignee: Olivier Thomann CLA
QA Contact:
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2002-12-11 10:31 EST by Channing Walton CLA
Modified: 2003-10-13 12:19 EDT (History)
1 user (show)

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 Channing Walton CLA 2002-12-11 10:31:40 EST 
I have some javadoc that somehow ended up like this:

/**
     * ssdfsd f
     * sd fsd fd 
     */

The formatter doesn't correct this to

/**
 * ssdfsd f
 * sd fsd fd 
 */
Comment 1 Olivier Thomann CLA 2002-12-13 13:57:08 EST 
Is the javadoc indented properly before formatting the code? If not, I don't see
why the formatter would do it. The user might want to leave the javadoc as is.
You don't even have to put '*' at the beginning of each line, so why the
formatter would take care of that?
I agree that the indentation has to be preserved during the formatting.
Comment 2 Channing Walton CLA 2002-12-14 05:23:15 EST 
ok, I didn't realise that the indentation of the '*' was ok. I had assumed that
they were meant to line up vertically. 

Perhaps this could be a feature request for a new preference ?

channing
Comment 3 Olivier Thomann CLA 2003-01-15 13:22:44 EST 
We need to clear 2.1 bug reports that won't be addressed before 2.1. The new 
implementation is still in the works. Therefore we cannot include it for 2.1. 
Not enough testing and we need to polish the preferences. This will be address 
for 2.2 as stated in the JDT/Core plan.
Comment 4 Philipe Mulet CLA 2003-06-12 06:36:12 EDT 
Resurrecting for 3.0
Comment 5 Olivier Thomann CLA 2003-06-12 15:59:02 EDT 
Reopen for 3.0 consideration.
Comment 6 Olivier Thomann CLA 2003-10-03 10:16:38 EDT 
This is addressed in the new formatter.
Fixed and released in HEAD.
Regression test added.
Comment 7 David Audel CLA 2003-10-13 12:19:08 EDT 
Verified.