Bug 22528 - Code formatter incorrectly indents lines
Summary: Code formatter incorrectly indents lines
Status: VERIFIED FIXED
Alias: None
Product: JDT
Classification: Eclipse Project
Component: Core (show other bugs)
Version: 2.0   Edit
Hardware: PC Windows 2000
: P3 normal (vote)
Target Milestone: 3.0 M4   Edit
Assignee: Olivier Thomann CLA
QA Contact:
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2002-08-19 10:59 EDT by Knut Radloff CLA
Modified: 2003-10-13 10:44 EDT (History)
0 users

See Also:

Attachments
Test case java file (29.46 KB, text/plain)
2002-08-19 14:39 EDT, Knut Radloff CLA 		no flags Details
View All 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 Knut Radloff CLA 2002-08-19 10:59:58 EDT 
build 20020815

-import the attached file CopyFilesAndFoldersOperation
-go to the copyFiles method
-select Source->Format

The first two lines in the WorkspaceModifyOperation are indented incorrectly.
My maximum line length is set to 120.
Comment 1 Dirk Baeumer CLA 2002-08-19 13:24:31 EDT 
Knut, the attachment is missing. And another question: what code has been 
selected before executing Fomat?
Comment 2 Knut Radloff CLA 2002-08-19 14:39:40 EDT 
Created attachment 1857 [details]
Test case java file
Comment 3 Knut Radloff CLA 2002-08-19 14:41:38 EDT 
I didn't have any code selected when I ran the code format.
Comment 4 Kai-Uwe Maetzel CLA 2002-09-04 03:57:18 EDT 
Core formatter issue.
Comment 5 Olivier Thomann CLA 2003-01-15 13:22:31 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-12 06:35:34 EDT 
Resurrecting for 3.0
Comment 7 Olivier Thomann CLA 2003-06-12 15:58:49 EDT 
Reopen for 3.0 consideration.
Comment 8 Olivier Thomann CLA 2003-10-03 10:04:38 EDT 
This is addressed in the new formatter.
Fixed and released in HEAD.
Regression test added.
Comment 9 David Audel CLA 2003-10-13 10:44:01 EDT 
Verified.