Re: [PATCH] D15445: clang-format: Fix WebKit BeforeElse brace wrapping style

2015-12-14 Thread Daniel Jasper via cfe-commits
djasper closed this revision. djasper added a comment. Submitted as r255484. http://reviews.llvm.org/D15445 ___ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Re: [PATCH] D15445: clang-format: Fix WebKit BeforeElse brace wrapping style

2015-12-11 Thread Cody Krieger via cfe-commits
cjk added a comment. Excellent. Mind committing this on my behalf? I don't have commit access. http://reviews.llvm.org/D15445 ___ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Re: [PATCH] D15445: clang-format: Fix WebKit BeforeElse brace wrapping style

2015-12-11 Thread Daniel Jasper via cfe-commits
djasper accepted this revision. djasper added a comment. This revision is now accepted and ready to land. Looks good. Funny that we don't seem to have a test for this. http://reviews.llvm.org/D15445 ___ cfe-commits mailing list cfe-commits@lists.llv

Re: [PATCH] D15445: clang-format: Fix WebKit BeforeElse brace wrapping style

2015-12-11 Thread Cody Krieger via cfe-commits
cjk added a comment. @djasper: Tagged you because it seems like you've reviewed a bunch recent clang-format patches. This is my first patch to any of the projects in the LLVM family, so please let me know if I've done something stupid, not added the proper reviewers/lists, etc. Cheers! http: