[
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)
