On 06.04.2013 4:00, Anna Zaks wrote:
On Apr 4, 2013, at 4:46 PM, Anton Yartsev <[email protected]
<mailto:[email protected]>> wrote:
+ // Used to suppress warnings if they are not related to the
tracked family
+ // (derived from AllocDeallocStmt).
The comment should focus on what the function does not what it's used
for. Also, the name is a bit misleading. isTrackedFamily sounds like
"are we currently tracking this family in the state?".
Also, let's try to stick to doxygen comments.
+ bool isTrackedFamily(AllocationFamily Family) const;
+ bool isTrackedFamily(CheckerContext &C, const Stmt
*AllocDeallocStmt) const;
+ bool isTrackedFamily(CheckerContext &C, SymbolRef Sym) const;
+
Refactored at r179242.
--
Anton
_______________________________________________
cfe-commits mailing list
[email protected]
http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits
