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

[PATCH] D14188: Format: support inline namespaces

2015-10-29 Thread Saleem Abdulrasool via cfe-commits
compnerd created this revision. compnerd added a reviewer: djasper. compnerd added a subscriber: cfe-commits. Herald added a subscriber: klimek. Correct handling for C++17 inline namespaces. We would previously fail to identify the inline namespaces as a namespace name since multiple ones may be