[PATCH] D135466: [clang-format] Add support to remove unnecessary semicolons after function definition
Jakub Klinkovský via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Fri Oct 7 11:56:38 PDT 2022
lahwaacz added inline comments.
================
Comment at: clang/docs/ClangFormatStyleOptions.rst:3769
+ care should be taken to review code changes made by this option.
+ NOTE:
+ Setting this to false will not add `;` where they were missing
----------------
Nit: the note should probably not be part of the warning.
================
Comment at: clang/docs/ClangFormatStyleOptions.rst:3779
+ return a>b?a:b; return a>b?a:b;
+ }; };
+
----------------
There is an extra `;` in the `true` case ;-)
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D135466/new/
https://reviews.llvm.org/D135466
More information about the cfe-commits
mailing list