[PATCH] D45719: [clang-Format] Fix indentation of member call after block

ank added a comment. ping @klimek Repository: rC Clang https://reviews.llvm.org/D45719 ___ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[PATCH] D45719: [clang-Format] Fix indentation of member call after block

ank added a comment. ping Repository: rC Clang https://reviews.llvm.org/D45719 ___ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[PATCH] D45719: [clang-Format] Fix indentation of member call after block

ank added a comment. Ping! it would be awesome to get some help getting this merged since I do not have merge rights Repository: rC Clang https://reviews.llvm.org/D45719 ___ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.

[PATCH] D45719: [clang-Format] Fix indentation of member call after block

ank updated this revision to Diff 158737. ank added a comment. fix space in if ( Repository: rC Clang https://reviews.llvm.org/D45719 Files: lib/Format/ContinuationIndenter.cpp lib/Format/FormatToken.h unittests/Format/FormatTest.cpp Index: unittests/Format/FormatTest.cpp ===

[PATCH] D45719: [clang-Format] Fix indentation of member call after block

ank added a comment. Herald added a subscriber: acoomans. Ping! it would be awesome to get some help getting this merged since I do not have merge rights Repository: rC Clang https://reviews.llvm.org/D45719 ___ cfe-commits mailing list cfe-comm

[PATCH] D45719: [clang-Format] Fix indentation of member call after block

ank added a comment. awesome, I do not have merge rights so help with merging this would be greatly appreciated Repository: rC Clang https://reviews.llvm.org/D45719 ___ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/c

[PATCH] D45719: [clang-Format] Fix indentation of member call after block

ank updated this revision to Diff 152870. Repository: rC Clang https://reviews.llvm.org/D45719 Files: lib/Format/ContinuationIndenter.cpp lib/Format/FormatToken.h unittests/Format/FormatTest.cpp Index: unittests/Format/FormatTest.cpp

[PATCH] D45719: [clang-Format] Fix indentation of member call after block

ank added inline comments. Comment at: unittests/Format/FormatTest.cpp:4449-4450 + " })\n" + " .foo(\"aaa\"\n" + " \"bb\");\n" +

[PATCH] D45719: [clang-Format] Fix indentation of member call after block

ank added inline comments. Comment at: unittests/Format/FormatTest.cpp:4359 + "return 3;\n" + " }).as("");\n" + "}"); klimek wrote: > ank wrote: > > klimek wrote: > > > ank wrote: > > > > klimek wrote: > > > > > What

[PATCH] D45719: [clang-Format] Fix indentation of member call after block

ank updated this revision to Diff 152842. Repository: rC Clang https://reviews.llvm.org/D45719 Files: lib/Format/ContinuationIndenter.cpp lib/Format/FormatToken.h unittests/Format/FormatTest.cpp Index: unittests/Format/FormatTest.cpp =

[PATCH] D45719: [clang-Format] Fix indentation of member call after block

ank added inline comments. Comment at: unittests/Format/FormatTest.cpp:4359 + "return 3;\n" + " }).as("");\n" + "}"); klimek wrote: > ank wrote: > > klimek wrote: > > > What would be interesting is tests that: > > >

[PATCH] D45719: [clang-Format] Fix indentation of member call after block

ank added a comment. Is there any chance to get this change or a similar one in so we get same behaviour as in release_40, even though it does not correct all of the problems? Repository: rC Clang https://reviews.llvm.org/D45719 ___ cfe-commits

[PATCH] D45719: [clang-Format] Fix indentation of member call after block

ank added inline comments. Comment at: unittests/Format/FormatTest.cpp:4359 + "return 3;\n" + " }).as("");\n" + "}"); klimek wrote: > What would be interesting is tests that: > a) have another value after the closing

[PATCH] D45719: [clang-Format] Fix indentation of member call after block

ank added a comment. ping Repository: rC Clang https://reviews.llvm.org/D45719 ___ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits