Bug 43545 - Code Formatter: Don't separate long "import" clause.
Summary: Code Formatter: Don't separate long "import" clause.
Status: VERIFIED FIXED
Alias: None
Product: JDT
Classification: Eclipse Project
Component: Core (show other bugs)
Version: 3.0   Edit
Hardware: All All
: P3 enhancement (vote)
Target Milestone: 3.0 M4   Edit
Assignee: Olivier Thomann CLA
QA Contact:
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2003-09-23 19:33 EDT by Alexander Ilyin CLA
Modified: 2003-10-14 11:14 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 Alexander Ilyin CLA 2003-09-23 19:33:02 EDT 
If I have long import clause exceed max line lenght, code formatter separate it
on many little lines. I think it's not readable.

Please add the option of the Code Formatter to turn off formatting for import lines
Comment 1 Olivier Thomann CLA 2003-09-24 07:15:38 EDT 
This is already done with the new formatter. We don't split anymore import
declarations no matter how long they can be.
Comment 2 Olivier Thomann CLA 2003-10-03 10:42:17 EDT 
This is addressed in the new formatter. The import statements are never split.
Fixed and released in HEAD.
Regression test added.
Comment 3 Alexander Ilyin CLA 2003-10-12 07:20:20 EDT 
I have installed 3.0M4. This feature seems doesn't work :-(
Comment 4 Olivier Thomann CLA 2003-10-14 09:07:35 EDT 
Did you activate the new formatter in the work in progress page?
Comment 5 Olivier Thomann CLA 2003-10-14 09:26:03 EDT 
Go to Preferences>Java>Work in progress and enable the new formatter.
Does it fix the problem?
Comment 6 Alexander Ilyin CLA 2003-10-14 11:06:24 EDT 
Yes. It's worked!
But it's very strange way and place to turn on this feature.
It's hard to guess it :-(
Comment 7 Olivier Thomann CLA 2003-10-14 11:14:19 EDT 
As long as there is no new code formatter preference page, the new code 
formatter is considered as a work in progress and is disabled by default. I am 
glad it works as expected.
Close as FIXED.