Bug 14659 - Align method arguments on line break
Summary: Align method arguments on line break
Status: VERIFIED FIXED
Alias: None
Product: JDT
Classification: Eclipse Project
Component: Core (show other bugs)
Version: 2.0   Edit
Hardware: PC Windows NT
: P3 enhancement (vote)
Target Milestone: 3.0 M4   Edit
Assignee: Olivier Thomann CLA
QA Contact:
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2002-04-25 14:32 EDT by Rob Grzywinski CLA
Modified: 2003-10-13 10:13 EDT (History)
0 users

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 Rob Grzywinski CLA 2002-04-25 14:32:51 EDT 
An option should be provided in the Code Generation preference as to the 
alignment of the method parameters.  This should apply to both reformatting and 
while manually entering text.

Current functionality (20020416 -- WinXP):

/**
 * Some method.
 */
public void someMethod(String argument1, String argument2,
String argument3)

Desired functionality:

/**
 * Some method.
 */
public void someMethod(String argument1, String argument2  [enter is pressed]
                       String argument3)  [the cursor moves under the 1st arg]
Comment 1 Martin Aeschlimann CLA 2002-04-26 10:02:13 EDT 
formatter
Comment 2 Philipe Mulet CLA 2002-04-29 13:42:50 EDT 
Will reconsider post 2.0
Comment 3 Philipe Mulet CLA 2002-07-25 06:30:46 EDT 
Resurrecting formatter issues in 2.1 stream.
Comment 4 Philipe Mulet CLA 2002-07-25 06:32:35 EDT 
Clearing resolution
Comment 5 Olivier Thomann CLA 2003-01-15 13:22:35 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 6 Philipe Mulet CLA 2003-06-13 04:52:12 EDT 
Reopening
Comment 7 Olivier Thomann CLA 2003-10-03 09:51:26 EDT 
This is addressed in the new formatter.
Fixed and released in HEAD.
Regression test added.
Comment 8 David Audel CLA 2003-10-13 10:13:38 EDT 
Verified.