[Lldb-commits] [PATCH] D131998: [LLDB][NFC] Suppress spurious static inspection warnings

Wed Aug 17 01:46:58 PDT 2022

labath added a comment.

In D131998#3728016 <https://reviews.llvm.org/D131998#3728016>, @fixathon wrote:

> In D131998#3727664 <https://reviews.llvm.org/D131998#3727664>, @clayborg wrote:
>
>> Can we use C++ comments instead of C comments?
>
> We can, and I'll make the change. Do you think it would be a good idea to keep the coverity annotations only as C-style comments (comments for the machine)  to disambiguate it from the regular comments (comments for the humans) ?

The llvm comment style is documented here <https://llvm.org/docs/CodingStandards.html#comment-formatting>. I don't think we should be making exceptions for coverity comments. (also, I wouldn't want to be having too many of these machine annotations around the codebase).

Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D131998/new/

https://reviews.llvm.org/D131998