Re: [PATCH] D11837: Add WebKit brace style configuration option.

2015-08-10 Thread Roman Kashitsyn via cfe-commits
This revision was automatically updated to reflect the committed changes. Closed by commit rL26: Add WebKit brace style configuration option. (authored by lifted). Changed prior to commit: http://reviews.llvm.org/D11837?vs=31660&id=31665#toc Repository: rL LLVM http://reviews.llvm.org/D

Re: [PATCH] D11837: Add WebKit brace style configuration option.

2015-08-10 Thread Roman Kashitsyn via cfe-commits
lifted added a comment. I had commit access earlier, but I haven't tried it for long time. I'll ask for help if there are any problems. http://reviews.llvm.org/D11837 ___ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi

Re: [PATCH] D11837: Add WebKit brace style configuration option.

2015-08-10 Thread Manuel Klimek via cfe-commits
klimek accepted this revision. klimek added a reviewer: klimek. klimek added a comment. Ah, awesome, looks good then. Do you have commit access or need one of us to land it? http://reviews.llvm.org/D11837 ___ cfe-commits mailing list cfe-commits@li

Re: [PATCH] D11837: Add WebKit brace style configuration option.

2015-08-10 Thread Roman Kashitsyn via cfe-commits
lifted added a comment. In http://reviews.llvm.org/D11837#220408, @klimek wrote: > Can you also fix the current code in getWebKitStyle? Yep, I've already done it. http://reviews.llvm.org/D11837 ___ cfe-commits mailing list cfe-commits@lists.llvm.

Re: [PATCH] D11837: Add WebKit brace style configuration option.

2015-08-10 Thread Manuel Klimek via cfe-commits
klimek added a comment. Can you also fix the current code in getWebKitStyle? http://reviews.llvm.org/D11837 ___ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits