View Issue Details [ Jump to Notes ]

[ Print ]

Project

Category

View Status

Date Submitted

Last Update

0011046

KWStyle

public

2010-07-27 13:51

Reporter

Brad Davis

Assigned To

Brad Davis

Priority

normal

Severity

minor

Reproducibility

have not tried

Status

assigned

Resolution

open

Platform

OS Version

Summary

0011046: Allow consistent brace placement

Description

Brace indentation checks differ for:
*class methods defined in class decl vs. outside class decl
*class decl inside function vs outside function
(others?...)

This is contrary to the ITK style rules and should be fixed (as an option?).

See additional information for specifics.

Additional Information

//
// new way; should pass
void Function()
{
  int i;
}

class C
{
public:
  void ClassMethod()
  { // new 2; old 4
    int i;
  } // new 2; old 4
};

void FunctionWithLocalClass()
{
  class LocalClass
  { // new 2; old 4
  public:
    void LocalClassMethod()
    { // new 4; old 6
      int i;
    } // new 4; old 6
  }; // new 2; old 4
}

//
// old way; should fail
void Function()
{
  int i;
}

class C
{
public:
  void ClassMethod()
    { // new 2; old 4
    int i;
    } // new 2; old 4
};

void FunctionWithLocalClass()
{
  class LocalClass
    { // new 2; old 4
  public:
      void LocalClassMethod()
      { // new 4; old 6
      int i;
      } // new 4; old 6
    }; // new 2; old 4
}

Issue History
Date Modified	Username	Field	Change
2010-07-27 13:51	Brad Davis	New Issue
2010-07-27 13:51	Brad Davis	Assigned To	=> Brad Davis
2010-07-27 13:51	Brad Davis	Status	new => assigned

Relationships

Notes
There are no notes attached to this issue.