galenelias wrote:

> Is there an github "Issue" for this? I can't quite understand what problem we 
> are trying to fix.

There is not a GitHub Issue filed yet, but I could get one filed if that would 
be helpful.

The issue is just there are users who want to use 
`AllowShortNamespacesOnASingleLine=true`, but who also currently use 
`BraceWrapping.AfterNamespace=true`.

So, they want the following code:

```cpp
namespace Foo
{
struct Bar;
}
```

To be formatted as:
```cpp
namespace Foo { struct Bar; }
```

But currently that isn't working.  AllowShortNamespacesOnASingleLine is 
currently written assuming BraceWrapping.AfterNamespace == false (which is the 
more common style).

https://github.com/llvm/llvm-project/pull/123010
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to