move HeaderSearch setup to Driver lib

Hi!
Is the topic related thing on the todo list ?
It would let users to build a driver more easily.
Currently i need this feature because i’m working on a IDE (codeblocks) plugin what parses the source file with clang and i dont want duplicate the clang.cpp’s setup code.

Csaba Hruska

Hi,

Is the topic related thing on the todo list ?
It would let users to build a driver more easily.

I agree. The attached patch moves most of the HeaderSearch setup to libDriver. Comments?

Nico

initinclude.patch (29.3 KB)

Is the topic related thing on the todo list ?
It would let users to build a driver more easily.

I agree. The attached patch moves most of the HeaderSearch setup to libDriver. Comments?

<initinclude.patch>

So, since no-one objects, can I check this in? :stuck_out_tongue:

Nico

Functionally the patch looks good to me. Could you add doxygen comments to the class InitHeaderSearch to describe what it does? From the way it is used, it is clearly a "builder object" that is used to populate data into a HeaderSearch object. It would be great if the doxygen comments above that class basically just said that. Other than that, it looks good to apply (assuming all tests pass).