[PATCH] D128402: [clang-tidy] Don't treat invalid branches as identical

Eugene Zelenko via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Thu Jun 23 06:39:56 PDT 2022


Eugene.Zelenko added inline comments.


================
Comment at: clang-tools-extra/docs/ReleaseNotes.rst:236
 
+- Fixed a false positive in :doc:`bugprone-branch-clone
+  <clang-tidy/checks/bugprone-branch-clone>` when the branches
----------------
ishaangandhi wrote:
> ishaangandhi wrote:
> > Eugene.Zelenko wrote:
> > > Documentation path was changed recently. Please also keep alphabetical order inside section.
> > Hm? Its still at this path on mastesr: https://github.com/llvm/llvm-project/blob/main/clang-tools-extra/docs/ReleaseNotes.rst
> Also, it doesn't look alphabetized to me? Two points up is "performance-unnecessary-..." then its "bugprone-use-after...".
Sorry, my mistake.


================
Comment at: clang-tools-extra/docs/ReleaseNotes.rst:236
 
+- Fixed a false positive in :doc:`bugprone-branch-clone
+  <clang-tidy/checks/bugprone-branch-clone>` when the branches
----------------
Eugene.Zelenko wrote:
> ishaangandhi wrote:
> > ishaangandhi wrote:
> > > Eugene.Zelenko wrote:
> > > > Documentation path was changed recently. Please also keep alphabetical order inside section.
> > > Hm? Its still at this path on mastesr: https://github.com/llvm/llvm-project/blob/main/clang-tools-extra/docs/ReleaseNotes.rst
> > Also, it doesn't look alphabetized to me? Two points up is "performance-unnecessary-..." then its "bugprone-use-after...".
> Sorry, my mistake.
It'll be good idea to fix order of other entries too.


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

https://reviews.llvm.org/D128402



More information about the cfe-commits mailing list