[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