NoQ added inline comments.

================
Comment at: clang/lib/StaticAnalyzer/Core/PlistDiagnostics.cpp:751
+        o << "    <key>insert_string</key>\n";
+        o << "    <string>" << Hint.CodeToInsert << "</string>\n";
+        o << "   </dict>\n";
----------------
gribozavr wrote:
> Escaping?
Oh #@&%! Thanks.


================
Comment at: clang/test/Analysis/dead-stores.c:11
   long idx=abc+3*5; // expected-warning {{never read}} 
expected-warning{{unused variable 'idx'}}
+  // expected-remark-re@-1{{{{.*}}:11 - {{.*}}:18 - ''}}
 }
----------------
gribozavr wrote:
> If the tests ignore the line number anyway, why even print it?
I anyway hope that this facility is temporary(c).


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

https://reviews.llvm.org/D65182



_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to