[PATCH] D37263: [clang-format] Ignore case when sorting using-declarations

2017-09-20 Thread Krasimir Georgiev via Phabricator via cfe-commits
krasimir updated this revision to Diff 115998. krasimir added a comment. - Stable sort using declarations https://reviews.llvm.org/D37263 Files: lib/Format/UsingDeclarationsSorter.cpp unittests/Format/UsingDeclarationsSorterTest.cpp Index: unittests/Format/UsingDeclarationsSorterTest.cpp =

[PATCH] D37263: [clang-format] Ignore case when sorting using-declarations

2017-09-19 Thread Alexander Kornienko via Phabricator via cfe-commits
alexfh added a comment. As discussed offline, the sorting should also be stable to avoid no-op replacements for identical using declarations and randomizing the order of using declarations differing only in case (not that I'd expect this to be a frequent thing, but it's better to handle it prop

[PATCH] D37263: [clang-format] Ignore case when sorting using-declarations

2017-09-08 Thread Krasimir Georgiev via Phabricator via cfe-commits
krasimir added a comment. ping https://reviews.llvm.org/D37263 ___ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[PATCH] D37263: [clang-format] Ignore case when sorting using-declarations

2017-08-29 Thread Krasimir Georgiev via Phabricator via cfe-commits
krasimir updated this revision to Diff 113092. krasimir added a comment. - Actually add a test https://reviews.llvm.org/D37263 Files: lib/Format/UsingDeclarationsSorter.cpp unittests/Format/UsingDeclarationsSorterTest.cpp Index: unittests/Format/UsingDeclarationsSorterTest.cpp ===

[PATCH] D37263: [clang-format] Ignore case when sorting using-declarations

2017-08-29 Thread Krasimir Georgiev via Phabricator via cfe-commits
krasimir created this revision. This ignores case while sorting using-declarations, fixing the wierd case in the last test case added. https://reviews.llvm.org/D37263 Files: lib/Format/UsingDeclarationsSorter.cpp unittests/Format/UsingDeclarationsSorterTest.cpp Index: unittests/Format/Usi