devnexen added a comment.

In, @NoQ wrote:

> All right, so the code looks good now, but in order to commit this, we also 
> need tests. We've got those automatic tests of ours in `test/Analysis/` that 
> feed small code snippets into the analyzer and verify that it does (or 
> doesn't) emit warnings in certain cases, so you'd need to prove that your 
> checker works (as intended, ideally) by writing some of those 
> (

Updated with a little new unit test.

  rC Clang

cfe-commits mailing list

Reply via email to