Hi Manuel,

On 28 sep. 2011, at 16:38, "Holtgrewe, Manuel" <[email protected]> 
wrote:

> Dear all,
> 
> attached is a patch that exposes more statement, expression and declaration 
> types through libclang. Also, I updated the Python bindings for these changes.
> 
> I have previously sent a patch but that has been overlooked so far. Thus, I 
> resubmit another patch, this time with [PATCH] in the subject and hope that 
> the missing [PATCH] was the reason  for why it was overlooked. If there are 
> other requirements to be met before submitting patches then please tell me so.
> 
> The attached patch is a superset of the previous patch.

Disclaimer: I am not the "module owner".

Feedback: looks good, but why change a hardcoded path?


--- lib/Frontend/InitHeaderSearch.cpp    (revision 140696)
+++ lib/Frontend/InitHeaderSearch.cpp    (working copy)
@@ -430,7 +430,7 @@
    break;
  default:
    // FIXME: temporary hack: hard-coded paths.
-    AddPath("/usr/local/include", System, true, false, false);
+    AddPath("/usr/include/c++/4.4", System, true, false, false);
    break;
  }

-- Erik
_______________________________________________
cfe-commits mailing list
[email protected]
http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits

Reply via email to