[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
Tue Nov 15 17:14:00 PST 2011



Sent from my iPhone

On Nov 10, 2011, at 4:58 PM, Chandler Carruth <chandlerc at google.com> 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?

*sigh* I just can't do it. My editor falls over itself and dies with the hanging brace, making the code uneditable. I'll be happy to argue for a tweak to the style guide :)
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20111115/e82a4fd2/attachment.html>


More information about the cfe-commits mailing list