<div dir="ltr">Hi Malcolm,<div><br></div><div>Thanks. I will fix the last couple of things in the weekend and hopefully have something worth showing there. </div><div><br></div><div>alexfh suggested that fixits seemed easy to implement. I am having a few doubts as to how I would make fixits for case 1 & 2. How important would it be to implement fixits at this point? </div><div><br></div><div>Best regards,</div><div>Mads Ravn</div></div><br><div class="gmail_quote"><div dir="ltr">On Fri, Dec 2, 2016 at 10:57 AM Malcolm Parsons <<a href="mailto:malcolm.parsons@gmail.com">malcolm.parsons@gmail.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">On 2 December 2016 at 09:50, Mads Ravn <<a href="mailto:madsravn@gmail.com" class="gmail_msg" target="_blank">madsravn@gmail.com</a>> wrote:<br class="gmail_msg">
>> Comment at: test/clang-tidy/misc-string-compare.cpp:9<br class="gmail_msg">
>> +  // CHECK-MESSAGES: [[@LINE-1]]:3: warning: do not use compare to test<br class="gmail_msg">
>> equality of strings; use the string equality operator instead<br class="gmail_msg">
>> [misc-string-compare]<br class="gmail_msg">
> What do you mean by this comment?<br class="gmail_msg">
<br class="gmail_msg">
I was replying to my previous line comment, but the line that was<br class="gmail_msg">
commented on has changed since.<br class="gmail_msg">
<br class="gmail_msg">
My comment was:<br class="gmail_msg">
<br class="gmail_msg">
There's still no test for the single parameter c-string overload.<br class="gmail_msg">
<br class="gmail_msg">
--<br class="gmail_msg">
Malcolm Parsons<br class="gmail_msg">
</blockquote></div>