NoQ added inline comments.

================
Comment at: clang/include/clang/StaticAnalyzer/Checkers/Checkers.td:455
 
+def StreamTesterChecker : Checker<"StreamTester">,
+  HelpText<"Add test functions to StreamChecker for test and debugging 
purposes.">,
----------------
This should go into the debug package. We don't want users to enable it 
accidentally by enabling the whole `unix` package.


================
Comment at: clang/lib/StaticAnalyzer/Checkers/StreamChecker.cpp:26-28
+#define ASSERT_STREAMSTATE_OPENED                                              
\
+  assert(SS->isOpened() &&                                                     
\
+         "Create of error node failed (only way to get this state)?");
----------------
`assertStreamStateOpened(SS);` isn't much harder to type. Can we make it a 
function please?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D75682



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

Reply via email to