[PATCH] D34623: [clang-format] Add a test for associative map proto buffer fields

2017-06-27 Thread Krasimir Georgiev via Phabricator via cfe-commits
This revision was automatically updated to reflect the committed changes. Closed by commit rL306386: [clang-format] Add a test for associative map proto buffer fields (authored by krasimir). Repository: rL LLVM https://reviews.llvm.org/D34623 Files: cfe/trunk/unittests/Format/FormatTestProt

[PATCH] D34623: [clang-format] Add a test for associative map proto buffer fields

2017-06-27 Thread Daniel Jasper via Phabricator via cfe-commits
djasper accepted this revision. djasper added a comment. This revision is now accepted and ready to land. Looks good. Thank you! https://reviews.llvm.org/D34623 ___ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/ma

[PATCH] D34623: [clang-format] Add a test for associative map proto buffer fields

2017-06-26 Thread Krasimir Georgiev via Phabricator via cfe-commits
krasimir updated this revision to Diff 103962. krasimir added a comment. - Add map declarations spanning multiple lines https://reviews.llvm.org/D34623 Files: unittests/Format/FormatTestProto.cpp Index: unittests/Format/FormatTestProto.cpp ===

[PATCH] D34623: [clang-format] Add a test for associative map proto buffer fields

2017-06-26 Thread Daniel Jasper via Phabricator via cfe-commits
djasper added a comment. Can you create a more interesting test case where the map definition spans multiple lines? Possibly use qualified names for the field types. https://reviews.llvm.org/D34623 ___ cfe-commits mailing list cfe-commits@lists.llv