[Analyzer] [PATCH] Adding file name to SimpleStreamChecker

Adam Schnitzer adamschn at umich.edu
Sun Apr 21 22:56:02 PDT 2013


Anna,

Got it, sorry about the mixup. I will go ahead and work in a separate file.
But did it look like I was on the right track for the diagnostics?

Adam

On Mon, Apr 22, 2013 at 1:20 AM, Anna Zaks <ganna at apple.com> wrote:

> Adam,
>
> Sorry if I was not 100% clear. We'd like to leave the
> SimpleStreamChecker.cpp file as is for reference purposes. You can either
> create a new file or replace StreamChecker.cpp with your checker.
>
> Thanks,
> Anna.
> On Apr 20, 2013, at 11:34 PM, Adam Schnitzer <adamschn at umich.edu> wrote:
>
> > This is my first patch for the SimpleStreamChecker. It improves
> diagnostics by adding the file name in the case of a resource leak. I did
> so by adding a std::string to the StreamState to hold the file name.
> >
> > Any feedback would be great.
> >
> > Adam
> > <SimpleStreamChecker.patch>
>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20130422/c4e78e81/attachment.html>


More information about the cfe-commits mailing list