Bug 23111 - format line wrapping should respect sun's coding conventions
Summary: format line wrapping should respect sun's coding conventions
Status: VERIFIED FIXED
Alias: None
Product: JDT
Classification: Eclipse Project
Component: Core (show other bugs)
Version: 2.0   Edit
Hardware: All All
: P3 enhancement (vote)
Target Milestone: 3.0 M7   Edit
Assignee: Olivier Thomann CLA
QA Contact:
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2002-09-03 09:26 EDT by Frederik Zimmer CLA
Modified: 2004-03-26 05:52 EST (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 Frederik Zimmer CLA 2002-09-03 09:26:45 EDT 
Form the coding conventions:
//DON'T USE THIS INDENTATION
if ((condition1 && condition2)
    || (condition3 && condition4)
    ||!(condition5 && condition6)) { //BAD WRAPS
    doSomethingAboutIt();            //MAKE THIS LINE EASY TO MISS
} 

//USE THIS INDENTATION INSTEAD
if ((condition1 && condition2)
        || (condition3 && condition4)
        ||!(condition5 && condition6)) {
    doSomethingAboutIt();
} 

When you use format you get the first result. The same for while, for, method-
declarations...
Comment 1 Kai-Uwe Maetzel CLA 2002-09-03 10:49:19 EDT 
Moving to JDT core.
Comment 2 Philipe Mulet CLA 2002-09-19 08:29:21 EDT 
New approach should indeed default to such a behavior.
Comment 3 Olivier Thomann CLA 2003-01-15 13:22:37 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:19 EDT 
Resurrecting for 3.0
Comment 5 Olivier Thomann CLA 2004-02-13 08:38:08 EST 
Reopen to close it.
Comment 6 Olivier Thomann CLA 2004-02-13 08:38:36 EST 
Fixed in 3.0 stream.
Comment 7 Frederic Fusier CLA 2004-03-26 05:52:07 EST 
Verified for 3.0 with build I200402122000.