Bug 98089 - Indent on Column alignment in formatter doesn't handle all conditions
Summary: Indent on Column alignment in formatter doesn't handle all conditions
Status: RESOLVED DUPLICATE of bug 72402
Alias: None
Product: JDT
Classification: Eclipse Project
Component: Core (show other bugs)
Version: 3.1   Edit
Hardware: PC Windows XP
: P3 normal (vote)
Target Milestone: ---   Edit
Assignee: JDT-Core-Inbox CLA
QA Contact:
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2005-06-02 08:17 EDT by Ian Brown CLA
Modified: 2005-08-31 10:37 EDT (History)
0 users

See Also:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Ian Brown CLA 2005-06-02 08:17:18 EDT
I believe this to be a regression. In the code formatter, you can specify Line 
Wrapping rules. I have a problem with the Constructor and Method 
Declarations|Paramters setting - and you can see the problem in the Preview...

If you set the Line wrapping policy to "Wrap all elements, except first...", 
and the Indentation policy to "Indent on column", the formatter attempts to 
align all parameters.

This formatting is *ignoring* the "space after open parenthesis" selection 
from the White Space tab - but correctly reacts to "space before open 
parenthesis"

This gives you the badly formatted preview of:

   void foo( int arg1,
            int arg2,
            int arg3,
            int arg4,
            int arg5,
            int arg6 )

This applies to Method and Constructor declarations - but the Function Calls 
equivalents seem to get it right
Comment 1 Olivier Thomann CLA 2005-06-02 09:33:03 EDT
This won't be fixed for 3.1
Comment 2 Olivier Thomann CLA 2005-08-30 13:57:04 EDT
Fixed and released in HEAD.
Regression test added in
org.eclipse.jdt.core.tests.formatter.FormatterRegressionTests.test592
Comment 3 Olivier Thomann CLA 2005-08-31 10:36:54 EDT
Reopen to close as duplicate of bug 72402.
Comment 4 Olivier Thomann CLA 2005-08-31 10:37:15 EDT

*** This bug has been marked as a duplicate of 72402 ***