Bug 10052 - CodeFormatter - line splitting enhancement.
Summary: CodeFormatter - line splitting enhancement.
Status: VERIFIED FIXED
Alias: None
Product: JDT
Classification: Eclipse Project
Component: Core (show other bugs)
Version: 2.0   Edit
Hardware: All All
: P3 normal with 2 votes (vote)
Target Milestone: 3.0 M4   Edit
Assignee: Olivier Thomann CLA
QA Contact:
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2002-02-21 08:32 EST by Jay Patel CLA
Modified: 2003-10-13 09:17 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 Jay Patel CLA 2002-02-21 08:32:12 EST 
Currently if I have line splitting set to 80 chars and if the current line
is longer than 80 chars, it is wrapped as follows:


      Logger.log(
        this,
        Logger.INFO,
        this.getFTPHost()
          + ": "
          + fromFile
          + " is less than 10 minutes old. Ignoring...");


As you can see that there is a lot of room on first five lines to
accommodate more than just one parameter on each line. Following is more 
desirable:


      Logger.log(this, Logger.INFO, this.getFTPHost() + ": "+ fromFile
               + " is less than 10 minutes old. Ignoring...");
Comment 1 Philipe Mulet CLA 2002-02-22 00:23:47 EST 
Personally, I prefer having parameters lined up once starting splitting.
Comment 2 Jay Patel CLA 2002-02-22 09:32:56 EST 
Can we allow for an option? Make the Formatter a bit more flexible where it can
allow different splittiting styles?

Thanx, 

Jay
Comment 3 Philipe Mulet CLA 2002-02-25 17:07:54 EST 
Formatter rework is planned post 2.0
Comment 4 Philipe Mulet CLA 2002-07-25 06:30:53 EDT 
Resurrecting formatter issues in 2.1 stream.
Comment 5 Philipe Mulet CLA 2002-07-25 06:32:42 EDT 
Clearing resolution
Comment 6 Olivier Thomann CLA 2003-01-15 13:22:49 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 7 Philipe Mulet CLA 2003-06-12 06:35:33 EDT 
Resurrecting for 3.0
Comment 8 Olivier Thomann CLA 2003-06-12 15:58:46 EDT 
Reopen for 3.0 consideration.
Comment 9 Olivier Thomann CLA 2003-10-03 09:47:32 EDT 
Line splitting should be improved in the new formatter.
Fixed and released in HEAD.
Comment 10 David Audel CLA 2003-10-13 09:17:18 EDT 
Verfied.