Re: [PATCH] D14188: Format: support inline namespaces

2015-10-30 Thread Saleem Abdulrasool via cfe-commits
compnerd closed this revision. compnerd added a comment. SVN r251690 with the mod. http://reviews.llvm.org/D14188 ___ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Re: [PATCH] D14188: Format: support inline namespaces

2015-10-29 Thread Daniel Jasper via cfe-commits
djasper accepted this revision. djasper added a comment. This revision is now accepted and ready to land. Expression can be simplified, otherwise looks good. Comment at: lib/Format/UnwrappedLineParser.cpp:1370 @@ -1369,2 +1369,3 @@ nextToken(); - if (FormatTok->Tok.is(tok::i