NoQ marked 5 inline comments as done. NoQ added inline comments.
================ Comment at: clang/lib/StaticAnalyzer/Checkers/StdLibraryFunctionsChecker.cpp:400 + /// then the return value is 0 + /// and the note may say "Assuming the character is non-alphabetical". + class SummaryCase { ---------------- balazske wrote: > It would be good to test how this looks in doxygen output. If it is for > generated documentation formatting tags can be used, otherwise this can be a > `//` comment. It doesn't! Classes in .cpp files don't seem to show up in doxygen at all. So it doesn't matter which style do we use. I think I'll keep it doxygenized just in case we decide to move these classes out to a public header at some point. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D122285/new/ https://reviews.llvm.org/D122285 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits