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
=
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
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
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
===
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