[clang] [clang-format] Support BraceWrapping.AfterNamespace with AllowShortNamespacesOnASingleLine (PR #123010)
Owen Pan via cfe-commits
cfe-commits at lists.llvm.org
Sun Jan 26 18:23:04 PST 2025
================
@@ -628,28 +632,36 @@ class LineJoiner {
unsigned tryMergeNamespace(ArrayRef<AnnotatedLine *>::const_iterator I,
ArrayRef<AnnotatedLine *>::const_iterator E,
- unsigned Limit) {
+ unsigned Limit, bool OpenBraceWrapped) {
if (Limit == 0)
return 0;
- assert(I[1]);
- const auto &L1 = *I[1];
+ // The merging code is relative to the opening namespace brace, which could
+ // be either on the first or second line due to the brace wrapping rules.
+ const size_t OpeningBraceLineOffset = OpenBraceWrapped ? 1 : 0;
----------------
owenca wrote:
```suggestion
const auto OpeningBraceLineOffset = OpenBraceWrapped ? 1 : 0;
```
to fix the [warning](https://github.com/llvm/llvm-project/pull/123010#issuecomment-2614727042).
https://github.com/llvm/llvm-project/pull/123010
More information about the cfe-commits
mailing list