[PATCH] D14170: Fix false positive warning about memory leak for QApplication::postEvent

Anna Zaks via cfe-commits cfe-commits at lists.llvm.org
Wed Nov 4 11:22:35 PST 2015


zaks.anna added a comment.

Test cases need to be small and self contained. 
You might be having a problem with reproducing with a simple test case, where you define QCoreApplication::postEvent in the test file because it is not considered to be in the system header.  You can use special pragmas for that. See ./test/Analysis/Inputs, where we have a bunch of .h files declaring system APIs. Please add another file for QT there.


================
Comment at: lib/StaticAnalyzer/Checkers/MallocChecker.cpp:2516
@@ +2515,3 @@
+    return true;
+  }
+
----------------
http://llvm.org/docs/CodingStandards.html#id16


http://reviews.llvm.org/D14170





More information about the cfe-commits mailing list