NoQ accepted this revision.
NoQ added inline comments.

================
Comment at: lib/StaticAnalyzer/Core/AnalyzerOptions.cpp:148-150
+  assert(Ret &&
+         "This option should be either 'true' or 'false', and should've been "
+         "validated by CheckerRegisrty!");
----------------
Even though `operator*()` would have asserted that there's a value, i really 
appreciate the user-friendly assert message.


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

https://reviews.llvm.org/D59195



_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
  • [PATCH] D59195: [analyzer]... Artem Dergachev via Phabricator via cfe-commits

Reply via email to