[jira] [Updated] (MESOS-8174) clang-format incorrectly indents aggregate initializations

["Benjamin Bannier (JIRA)" <ji...@apache.org>]

     [ https://issues.apache.org/jira/browse/MESOS-8174?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Benjamin Bannier updated MESOS-8174:
------------------------------------
    Description: 
Aggregate initializations are incorrectly indented. I would expect the following indention,

{code}
Foo bar{
    123,
    456,
    789};
{code}

Instead this is indented as

{code}
Foo bar{123,
        456,
        789};
{code}

Forcing a line break after the opening curly incorrectly indents the arguments with two instead of four spaces,

{code}
Foo bar{

  123,
  456,
  789};
{code}

The [Google C++ style guide|https://google.github.io/styleguide/cppguide.html#Braced_Initializer_List_Format] suggests to
{quote}
Format a braced initializer list exactly like you would format a function call in its place.
{quote}
and our style guide demands
{quote}
Newline when calling or defining a function: indent with four spaces.
{quote}

  was:
Aggregate initializations are incorrectly indented. I would expect the following indention,

{code}
Foo bar{
    123,
    456,
    789};
{code}

Instead this is indented as

{code}
Foo bar{123,
        456,
        789};
{code}

Forcing a line break after the opening curly incorrectly indents the arguments with two instead of four spaces,

{code}
Foo bar{

  123,
  456,
  789};
{code}


> clang-format incorrectly indents aggregate initializations
> ----------------------------------------------------------
>
>                 Key: MESOS-8174
>                 URL: https://issues.apache.org/jira/browse/MESOS-8174
>             Project: Mesos
>          Issue Type: Bug
>            Reporter: Benjamin Bannier
>              Labels: clang-format
>
> Aggregate initializations are incorrectly indented. I would expect the following indention,
> {code}
> Foo bar{
>     123,
>     456,
>     789};
> {code}
> Instead this is indented as
> {code}
> Foo bar{123,
>         456,
>         789};
> {code}
> Forcing a line break after the opening curly incorrectly indents the arguments with two instead of four spaces,
> {code}
> Foo bar{
>   123,
>   456,
>   789};
> {code}
> The [Google C++ style guide|https://google.github.io/styleguide/cppguide.html#Braced_Initializer_List_Format] suggests to
> {quote}
> Format a braced initializer list exactly like you would format a function call in its place.
> {quote}
> and our style guide demands
> {quote}
> Newline when calling or defining a function: indent with four spaces.
> {quote}



--
This message was sent by Atlassian JIRA
(v6.4.14#64029)