[PATCH] Add scan-build python implementation

Daniel Marjamäki daniel.marjamaki at evidente.se
Fri May 8 05:28:11 PDT 2015


Nice!

this is awesome.

I can see that you test with travis. Did you consider to test it with appveyor also to get windows testing also?

ear.c:

- not written in C++, doesn't seem to me it conforms to LLVM/CLANG coding style
- As far as I know, __attribute__ is not ANSI C.. and can't be compiled by some compilers.
- unistd.h is a posix header isn't it? can't be used by some compilers.


================
Comment at: tools/scan-build2/libscanbuild/clang.py:8
@@ +7,3 @@
+
+Since Clang command line interface is so reach, but this project is using only
+a subset of that, it makes sense to create a function specific wrapper. """
----------------
reach => rich?

http://reviews.llvm.org/D9600

EMAIL PREFERENCES
  http://reviews.llvm.org/settings/panel/emailpreferences/






More information about the cfe-commits mailing list