<div dir="ltr">Thanks for fixing these.  Feel free to commit trivial warning fixes without review.</div><br><div class="gmail_quote"><div dir="ltr">On Wed, Aug 26, 2015 at 4:43 PM Stephane Sezer <<a href="mailto:sas@cd80.net">sas@cd80.net</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">sas created this revision.<br>
sas added a reviewer: zturner.<br>
sas added a subscriber: lldb-commits.<br>
<br>
Just `assert("string" && false)` instead of `assert("string" == NULL)`.<br>
<br>
This avoid errors like<br>
<br>
    [...]\Core\SourceManager.cpp(647): warning C4130: '==' : logical operation on address of string constant<br>
<br>
<a href="http://reviews.llvm.org/D12388" rel="noreferrer" target="_blank">http://reviews.llvm.org/D12388</a><br>
<br>
Files:<br>
  source/Core/SourceManager.cpp<br>
<br>
Index: source/Core/SourceManager.cpp<br>
===================================================================<br>
--- source/Core/SourceManager.cpp<br>
+++ source/Core/SourceManager.cpp<br>
@@ -644,14 +644,14 @@<br>
         else<br>
         {<br>
             // Some lines have been populated, start where we last left off<br>
-            assert("Not implemented yet" == NULL);<br>
+            assert("Not implemented yet" && false);<br>
         }<br>
<br>
     }<br>
     else<br>
     {<br>
         // Calculate all line offsets up to "line"<br>
-        assert("Not implemented yet" == NULL);<br>
+        assert("Not implemented yet" && false);<br>
     }<br>
     return false;<br>
 }<br>
<br>
<br>
</blockquote></div>