Bug 188007 - Namespace indenting preference is not honored
Summary: Namespace indenting preference is not honored
Status: RESOLVED FIXED
Alias: None
Product: CDT
Classification: Tools
Component: cdt-core (show other bugs)
Version: 4.0   Edit
Hardware: PC Linux
: P3 normal (vote)
Target Milestone: 4.0 RC3   Edit
Assignee: Anton Leherbauer CLA
QA Contact:
URL:
Whiteboard:
Keywords: contributed
Depends on:
Blocks:
 
Reported: 2007-05-20 18:57 EDT by Sergey Prigogin CLA
Modified: 2009-01-09 15:05 EST (History)
0 users

See Also:

Attachments
Proposed fix (6.80 KB, patch)
2007-05-26 20:26 EDT, Sergey Prigogin CLA 		bjorn.freeman-benson: iplog+
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 Sergey Prigogin CLA 2007-05-20 18:57:11 EDT 
CDT HEAD:
1. Uncheck Indent Declarations within 'namespace' definition checkbox in Code Style.
2. In C++ editor type "namespace test {" and press Enter.

The cursor is positioned one indentation level deeper compared to the namespace. It should be positioned at the same level.
Comment 1 Sergey Prigogin CLA 2007-05-26 20:26:27 EDT 
Created attachment 68863 [details]
Proposed fix

In addition to a fix for the indenting problem I adjusted namespace template to be consistent with the default namespace indenting, which is off.

I also took liberty to adjust class template, which has nothing to do with this bug.

Please review and apply.
Comment 2 Anton Leherbauer CLA 2007-05-29 03:04:33 EDT 
Thanks Sergey. Patch looks good. I'll add a test case.
Comment 3 Anton Leherbauer CLA 2007-05-29 03:38:01 EDT 
Patch applied.
Created test case CAutoIndentTest.testIndentInsideNamespaceDefinition_Bug188007