]> git.eshelyaron.com Git - emacs.git/commitdiff
Use the new string-collate-lessp function in ls-lisp.el.
authorEli Zaretskii <eliz@gnu.org>
Mon, 1 Sep 2014 14:57:21 +0000 (17:57 +0300)
committerEli Zaretskii <eliz@gnu.org>
Mon, 1 Sep 2014 14:57:21 +0000 (17:57 +0300)
 lisp/ls-lisp.el (ls-lisp-use-string-collate)
 (ls-lisp-UCA-like-collation): New defcustoms.
 (ls-lisp-string-lessp): Use them to control sorting by file
 names.

 etc/NEWS: Mention that ls-lisp uses string-collate-lessp.

Fixes: debbugs:18051
etc/ChangeLog
etc/NEWS
lisp/ChangeLog
lisp/ls-lisp.el

index ff00659cba0f26dbd1cf3cdb68b7d0676d2c3265..8dbdb46c826e323b7ed8e6b6c79101ba2e563aa2 100644 (file)
@@ -1,3 +1,7 @@
+2014-09-01  Eli Zaretskii  <eliz@gnu.org>
+
+       * NEWS: Mention that ls-lisp uses string-collate-lessp.
+
 2014-09-01  Paul Eggert  <eggert@cs.ucla.edu>
 
        --enable-silent-rules now suppresses more chatter.
index 100214f1108bc93f882bfc66866d461248ca5ffd..1d9f8ca850f9cad9e42a7fcd2afe81212d63b4a5 100644 (file)
--- a/etc/NEWS
+++ b/etc/NEWS
@@ -75,6 +75,10 @@ environment.  For the time being this is implemented for modern POSIX
 systems and for MS-Windows, for other systems they fall back to their
 counterparts `string-lessp' and `string-equal'.
 
+*** The ls-lisp package uses `string-collate-lessp' to sort file names.
+If you want the old, locale-independent sorting, customize the new
+option `ls-lisp-use-string-collate' to a nil value.
+
 *** The MS-Windows specific variable `w32-collate-ignore-punctuation',
 if set to a non-nil value, causes the above 2 functions to ignore
 symbol and punctuation characters when collating strings.  This
index 7087cde6d44102b5ba4cf713b7bb50df0aa79b63..47185e01f5e4a4e1a369b986a6b507b8f78f47ec 100644 (file)
@@ -1,3 +1,10 @@
+2014-09-01  Eli Zaretskii  <eliz@gnu.org>
+
+       * ls-lisp.el (ls-lisp-use-string-collate)
+       (ls-lisp-UCA-like-collation): New defcustoms.
+       (ls-lisp-string-lessp): Use them to control sorting by file
+       names.  (Bug#18051)
+
 2014-08-31  Christoph Scholtes  <cschol2112@gmail.com>
 
        * ibuffer.el: Replace mode-specific quit function with
index 2ff0a3a230b6b1831402e9198c378a812fe7f91c..410b44999f614e01cd58da34303a40c90bec9d44 100644 (file)
@@ -113,6 +113,47 @@ update the dependent variables."
   :type 'boolean
   :group 'ls-lisp)
 
+(defcustom ls-lisp-use-string-collate
+  (cond ((memq ls-lisp-emulation '(MacOS UNIX)) nil)
+       (t t))          ; GNU/Linux or MS-Windows emulate GNU ls
+  "Non-nil causes ls-lisp to sort files in locale-dependent collation order.
+
+A value of nil means use ordinary string comparison (see `compare-strings')
+for sorting files.  A non-nil value uses `string-collate-lessp' instead,
+which more closely emulates what GNU `ls' does.
+
+On GNU/Linux systems, if the locale's codeset specifies UTF-8, as
+in \"en_US.UTF-8\", the collation order follows the Unicode
+Collation Algorithm (UCA), which places together file names that
+differ only in punctuation characters.  On MS-Windows, customize
+the option `ls-lisp-UCA-like-collation' to a non-nil value to get
+similar behavior."
+  :version "24.5"
+  :set-after '(ls-lisp-emulation)
+  :type 'boolean
+  :group 'ls-lisp)
+
+(defcustom ls-lisp-UCA-like-collation t
+  "Non-nil means force ls-lisp use a collation order compatible with UCA.
+
+UCA is the Unicode Collation Algorithm.  GNU/Linux systems automatically
+follow it in their string-collation routines if the locale specifies
+UTF-8 as its codeset.  On MS-Windows, customize this option to a non-nil
+value to get similar behavior.
+
+When this option is non-nil, and `ls-lisp-use-string-collate' is also
+non-nil, the collation order produced on MS-Windows will ignore
+punctuation and symbol characters, which will, for example, place
+\`.foo' near `foo'.  See the documentation of `string-collate-lessp'
+and `w32-collate-ignore-punctuation' for more details.
+
+This option is ignored on platforms other than MS-Windows; to
+control the collation ordering of the file names on those other
+systems, set your locale instead."
+  :version "24.5"
+  :type 'boolean
+  :group 'ls-lisp)
+
 (defcustom ls-lisp-dirs-first (eq ls-lisp-emulation 'MS-Windows)
   "Non-nil causes ls-lisp to sort directories first in any ordering.
 \(Or last if it is reversed.)  Follows Microsoft Windows Explorer."
@@ -495,11 +536,20 @@ Responds to the window width as ls should but may not!"
     result))
 
 (defsubst ls-lisp-string-lessp (s1 s2)
-  "Return t if string S1 is less than string S2 in lexicographic order.
+  "Return t if string S1 should sort before string S2.
 Case is significant if `ls-lisp-ignore-case' is nil.
-Unibyte strings are converted to multibyte for comparison."
-  (let ((u (compare-strings s1 0 nil s2 0 nil ls-lisp-ignore-case)))
-    (and (numberp u) (< u 0))))
+Uses `string-collate-lessp' if `ls-lisp-use-string-collate' is non-nil,
+\`compare-strings' otherwise.
+On GNU/Linux systems, if the locale specifies UTF-8 as the codeset,
+the sorting order will place together file names that differ only
+by punctuation characters, like `.emacs' and `emacs'.  To have a
+similar behavior on MS-Widnows, customize `ls-lisp-UCA-like-collation'
+to a non-nil value."
+  (let ((w32-collate-ignore-punctuation ls-lisp-UCA-like-collation))
+    (if ls-lisp-use-string-collate
+       (string-collate-lessp s1 s2 nil ls-lisp-ignore-case)
+      (let ((u (compare-strings s1 0 nil s2 0 nil ls-lisp-ignore-case)))
+       (and (numberp u) (< u 0))))))
 
 (defun ls-lisp-handle-switches (file-alist switches)
   "Return new FILE-ALIST sorted according to SWITCHES.