Michael137 wrote:
Getting following warning with this patch:
```
/Users/jonas/Git/llvm-worktrees/llvm-project/lldb/unittests/Core/MangledTest.cpp:417:40:
warning: missing field 'PrefixRange' initializer [-Wmissing-field-initializers]
417 | /*.QualifiersRange=*/{158, 176} },
| ^
/Users/jonas/Git/llvm-worktrees/llvm-project/lldb/unittests/Core/MangledTest.cpp:423:122:
warning: missing field 'PrefixRange' initializer [-Wmissing-field-initializers]
423 | { /*.BasenameRange=*/{6, 13}, /*.ScopeRange=*/{6, 6},
/*.ArgumentsRange=*/{ 20, 27 }, /*.QualifiersRange=*/{38, 38} },
|
^
/Users/jonas/Git/llvm-worktrees/llvm-project/lldb/unittests/Core/MangledTest.cpp:430:38:
warning: missing field 'PrefixRange' initializer [-Wmissing-field-initializers]
430 | /*.QualifiersRange=*/{12, 12} },
| ^
/Users/jonas/Git/llvm-worktrees/llvm-project/lldb/unittests/Core/MangledTest.cpp:437:38:
warning: missing field 'PrefixRange' initializer [-Wmissing-field-initializers]
437 | /*.QualifiersRange=*/{58, 58} },
| ^
/Users/jonas/Git/llvm-worktrees/llvm-project/lldb/unittests/Core/MangledTest.cpp:444:38:
warning: missing field 'PrefixRange' initializer [-Wmissing-field-initializers]
444 | /*.QualifiersRange=*/{79, 79} },
| ^
/Users/jonas/Git/llvm-worktrees/llvm-project/lldb/unittests/Core/MangledTest.cpp:451:40:
warning: missing field 'PrefixRange' initializer [-Wmissing-field-initializers]
451 | /*.QualifiersRange=*/{168, 168} },
| ^
```
when compiling the unit-tests.
Could you add the missing initializers to the test? I think you can just add a
default value so you don't need to touch every test-case
https://github.com/llvm/llvm-project/pull/140762
_______________________________________________
lldb-commits mailing list
lldb-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits
