Bug 49081 - JDT is no more using the tab character by default for indentation
Summary: JDT is no more using the tab character by default for indentation
Status: VERIFIED FIXED
Alias: None
Product: JDT
Classification: Eclipse Project
Component: Core (show other bugs)
Version: 3.0   Edit
Hardware: PC Linux
: P3 normal (vote)
Target Milestone: 3.0 M6   Edit
Assignee: Jerome Lanneluc CLA
QA Contact:
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2003-12-17 16:55 EST by Luc Bourlier CLA
Modified: 2003-12-18 12:00 EST (History)
0 users

See Also:

Attachments
Proposed patch (1.01 KB, patch)
2003-12-18 06:11 EST, Jerome Lanneluc CLA 		no flags Details | Diff
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 Luc Bourlier CLA 2003-12-17 16:55:50 EST 
I20031216

By default, using the tab key inserts space characters. Prior to the last few
Integration builds, Eclipse has always inserted tab character by default.
(I'm using the new code formatter)
Comment 1 Dirk Baeumer CLA 2003-12-17 17:12:21 EST 
In a fresh workspace with the new code formatter enabled, a tab key is 
inserted when pressing a tab key. Have you create your own code formatter 
profile ? If so, what value has the Use tabe character field on the Others 
page of the code formatter.

Creating a new profile initialize this field to unchecked. Default should be 
checked.
Comment 2 Dirk Baeumer CLA 2003-12-18 04:36:48 EST 
Last comment was wrong. In a fresh workspace, when using the new code 
formatter with either the Default or Java convention profile tab inserts 4 
spaces when used for indenting.

The Default profile is meant to be compatible with the old formatter so it 
should use tabs instead spaces (e.g the option "Use tab character" under the 
other tab should be checked).

Moving to JDT/Core since they provide the default profile.
Comment 3 Dirk Baeumer CLA 2003-12-18 04:39:36 EST 
We should consider this for M6 since it is really annoying when switching to 
the new formatter.
Comment 4 Jerome Lanneluc CLA 2003-12-18 06:11:02 EST 
Created attachment 7223 [details]
Proposed patch
Comment 5 Dirk Baeumer CLA 2003-12-18 06:19:17 EST 
Jerome, I tested it and it works for me.
Comment 6 Jerome Lanneluc CLA 2003-12-18 06:52:14 EST 
Released proposed patch and changed corresponding formatter tests.
Comment 7 Frederic Fusier CLA 2003-12-18 12:00:04 EST 
Verified for 3.0 M6 with build 200312180955.