| Issue |
142888
|
| Summary |
[clang-doc] Reconsider use of enum InfoType::IT_default
|
| Labels |
new issue
|
| Assignees |
|
| Reporter |
evelez7
|
The use of `InfoType::IT_default` seems to clash with LLVM style which [prefers to warn if not all enums are covered in a switch without `default`](https://llvm.org/docs/CodingStandards.html#don-t-use-default-labels-in-fully-covered-switches-over-enumerations). We use our own default enum, which a.) could just be covered by `default` and b.) might prevent us from seeing the warning.
It doesn't look like we set it anywhere besides when default constructing an `Info`, so I doubt that it's actually checked before a proper type is set. It's not checked in the unit tests where it is explicitly passed to construct a reference.
![]()
_______________________________________________
llvm-bugs mailing list
llvm-bugs@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-bugs
