[cfe-commits] r144347 - in /cfe/trunk: include/clang/Lex/HeaderSearch.h lib/Frontend/ASTUnit.cpp lib/Frontend/CompilerInstance.cpp lib/Lex/HeaderSearch.cpp

Douglas Gregor dgregor at apple.com
Thu Nov 10 16:59:20 PST 2011


On Nov 10, 2011, at 4:58 PM, Chandler Carruth wrote:

> Style nit:
> 
> On Thu, Nov 10, 2011 at 4:35 PM, Douglas Gregor <dgregor at apple.com> wrote:
> +HeaderSearch::HeaderSearch(FileManager &FM, DiagnosticsEngine &Diags)
> +  : FileMgr(FM), Diags(Diags), FrameworkMap(64)
> +{
> 
> Don't the '{'s usually go on the line above?

I… find it insanely ugly when the "{" is handling there, and the next line of code lines up with the ":"

If you nag me, I'll hold my nose and move it up onto the line above.

	- Doug
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20111110/fdd0450b/attachment.html>


More information about the cfe-commits mailing list