Bug 188007

Summary: Namespace indenting preference is not honored
Product: [Tools] CDT Reporter: Sergey Prigogin <eclipse.sprigogin>
Component: cdt-coreAssignee: Anton Leherbauer <aleherb+eclipse>
Status: RESOLVED FIXED QA Contact:
Severity: normal    
Priority: P3 Keywords: contributed
Version: 4.0   
Target Milestone: 4.0 RC3   
Hardware: PC   
OS: Linux   
Whiteboard:
Attachments:
Description Flags
Proposed fix bjorn.freeman-benson: iplog+

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