[cfe-commits] [patch] Refactor the include search into different c, c++ and framework functions

[Rafael Espindola]

Ping.

2009/10/22 Rafael Espindola <espindola at google.com>:
> The attached patch refactors AddDefaultSystemIncludePaths by adding
> helper functions for c, c++ and framework. The only user visible
> differences should be
>
> *) Frameworks are only searched on OS X
> *) The Cygwin c++ headers are now marked as c++ aware. I am almost
> sure that not marking them was a bug.
>
> One reason for doing this is just that the current functions was
> getting large and with lots of nesting. Another, probably more
> important, reason is that I want to add support for defining search
> paths at compile time. With this refactoring this will look like
>
> ...AddDefaultCIncludePaths(..) {
>  If (configure option for C set)  {
>   use it.
>   return;
>   }
>  current run time search.
> }
>
> And similarly for c++ and maybe frameworks (if those two paths are not
> the only ones ever used....).
>
> Comments?
>
> Cheers,
> --
> Rafael Ávila de Espíndola
>



-- 
Rafael Ávila de Espíndola