[PATCH] D39121: [clang-tidy] Misplaced Operator in Strlen in Alloc

Aaron Ballman via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Mon Nov 6 07:28:25 PST 2017


aaron.ballman accepted this revision.
aaron.ballman added a comment.
This revision is now accepted and ready to land.

LGTM!



================
Comment at: test/clang-tidy/bugprone-misplaced-operator-in-strlen-in-alloc.c:1
+// RUN: %check_clang_tidy %s bugprone-misplaced-operator-in-strlen-in-alloc %t
+
----------------
baloghadamsoftware wrote:
> aaron.ballman wrote:
> > Please clang-format this file.
> I omitted it intantionally. No I did it, but it meant lots of editing back the `CHECK-MESSAGES` and `CHECK-FIXES` comments. They were broken were they should not have been, and merged where they should not have been.
Ugh, it's unfortunate that clang-format messed with the CHECK-* stuff, but thank you for ensuring the code is formatted according to the community standards.


https://reviews.llvm.org/D39121





More information about the cfe-commits mailing list