[cfe-dev] [Patch] Update to Checker Development Manual

Sam Handler samuel.handler+cfedev at gmail.com
Sun Apr 28 16:41:19 PDT 2013


On Wed, Apr 24, 2013 at 1:51 PM, Anna Zaks <ganna at apple.com> wrote:

> Sam,
>
> This looks very good! I have a couple of minor comments below. I would
> also like to have the CustomProgram States section move before the commit.
>
> --- Idea for a Checker
> We could briefly introduce the SimpleStreamChecker and what it does at the
> end of Idea for a Checker section. What do you think?
>

I'm not sure about how well it fits there. I'm not sure exactly how such an
introduction would be written, so I think it would be best to come back to
it later.

All of your other comments should be addressed. If you don't see any other
issues, please feel free to commit for me (as I don't have commit access).

--Sam


-- 
===============================================================================
All opinions expressed in posts from this account are entirely my own, and
not
those of any present or former employer. Furthermore, I assert that all
works
contributed to the Clang project (1) were developed using no equipment,
supplies, facility or trade secrets of any such employer, (2) were developed
entirely on my own time, and (3) do not result from any work performed for
any
such employer.
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20130428/d41c7cd7/attachment.html>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: checker_dev_manual.diff
Type: application/octet-stream
Size: 19599 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20130428/d41c7cd7/attachment.obj>


More information about the cfe-commits mailing list