-
Notifications
You must be signed in to change notification settings - Fork 120
Description
I'm running Subclipse 4.2.2 in Eclipse Neon 4.6.3 on OpenSUSE Leap 42.2. The SVN Repositories list sorts the repositories using a case-sensitive compare. Therefore the repos are sorted like follows:
http://server/NewRepo1
http://server/OldRepo1
http://server/newRepo2
http://server/oldRepo2
I think the code should be updated to use a case-insensitive compare so that the repos are sorted like follows:
http://server/NewRepo1
http://server/newRepo2
http://server/OldRepo1
http://server/oldRepo2
I don't know for sure, but I believe that changing compareTo to compareToIgnoreCase in https://github.com/subclipse/subclipse/blob/master/bundles/subclipse.ui/src/org/tigris/subversion/subclipse/ui/repository/RepositorySorter.java would fix this issue.