zaks.anna added inline comments.

================
Comment at: tools/scan-build-py/bin/analyze-c++:2
@@ +1,3 @@
+#!/usr/bin/env python
+# -*- coding: utf-8 -*-
+#                     The LLVM Compiler Infrastructure
----------------
Where/How is analyze-c++ used?

================
Comment at: tools/scan-build-py/bin/analyze-cc:14
@@ +13,2 @@
+from libscanbuild.analyze import wrapper
+sys.exit(wrapper(False))
----------------
I believe this is an important point. Fixing this would greatly improve 
readability!

================
Comment at: tools/scan-build-py/libear/ear.c:282
@@ +281,3 @@
+    DLSYM(func, fp, "execve");
+
+    char const **const menvp = bear_update_environment(envp, &initial_env);
----------------
rizsotto.mailinglist wrote:
> okay, since i don't have access for OSX machines can you help me to make it 
> right?
Ok. Will do.


http://reviews.llvm.org/D9600



_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to