<div>This is a small patch (my first) which I'm using to test the water for some other changes in the same vein.</div><div><br></div><div>Basically, the Parser constructor was taking a spurious Preprocessor parameter because the Sema already has a Preprocessor. This is a bug waiting to happen if a user of this API passes a Preprocessor different from the one owned by the Sema.</div>
<div><br></div><div>As I am using Clang's APIs for one-off tools (and binding to Lua), I found a similar situation occurring in a couple other places. The red arrows in this graph represent the spurious constructor parameters of other classes <a href="http://web.ics.purdue.edu/~silvas/deps.svg">http://web.ics.purdue.edu/~silvas/deps.svg</a> . If this patch gets good feedback I will fix up the other "red arrows" and submit another patch. Hopefully this will simplify the API for making one-off tools (which is currently pretty hairy and hard to use).</div>
<div><br></div><div>I basically changed the constructor and then fixed the compilation errors that resulted from the signature change.</div><div><br></div><div>Note: patch produced by `git diff`.</div><div><br></div><div>
--Sean Silva</div>