<div dir="ltr">Hi Malcolm,<div><br></div><div>Thanks for the suggestions, I have been reading up on the fixits. </div><div>My initial four cases has been reduced to two a little more general cases:</div><div>1 & 2: implicitCast to bool str1.compare(str2). This case covers both !str1.compare(str2) and str1.compare(str2)</div><div>3 & 4: str1.compare(str2) == 0 and str1.compare(str2) != 0. </div><div><br></div><div>I see the idea for the fixit clearly for case 3 & 4. Just erase .compare(str2) and replace 0 with str2. I have a quick question though: Given the declRefExpr().bind("str2"), how do I read the name of it in clang-tidy? Or should I just bind 0 as well and then create replacement with str where const auto str = Result.Nodes.getNodeAs<Stmt>("str2") ? </div><div><br></div><div>Where I seem to find a little trouble is how to fixit case 1 & 2 now that they are reduced to one case. How do I check whether or not there is a unary operator in front of the implicitCast? </div><div><br></div><div>Thank you,</div><div>Mads Ravn</div></div><br><div class="gmail_quote"><div dir="ltr">On Thu, Dec 1, 2016 at 8:53 PM Mads Ravn via Phabricator <<a href="mailto:reviews@reviews.llvm.org">reviews@reviews.llvm.org</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">madsravn updated this revision to Diff 79961.<br class="gmail_msg">
madsravn added a comment.<br class="gmail_msg">
<br class="gmail_msg">
Fixed broken tests.<br class="gmail_msg">
<br class="gmail_msg">
<br class="gmail_msg">
<a href="https://reviews.llvm.org/D27210" rel="noreferrer" class="gmail_msg" target="_blank">https://reviews.llvm.org/D27210</a><br class="gmail_msg">
<br class="gmail_msg">
Files:<br class="gmail_msg">
  clang-tidy/misc/CMakeLists.txt<br class="gmail_msg">
  clang-tidy/misc/MiscTidyModule.cpp<br class="gmail_msg">
  clang-tidy/misc/StringCompareCheck.cpp<br class="gmail_msg">
  clang-tidy/misc/StringCompareCheck.h<br class="gmail_msg">
  docs/ReleaseNotes.rst<br class="gmail_msg">
  docs/clang-tidy/checks/list.rst<br class="gmail_msg">
  docs/clang-tidy/checks/misc-string-compare.rst<br class="gmail_msg">
  test/clang-tidy/misc-string-compare.cpp<br class="gmail_msg">
<br class="gmail_msg">
</blockquote></div>