[Emacs-diffs] /srv/bzr/emacs/trunk r105542: Fix default values of character properties.

[Kenichi Handa]

------------------------------------------------------------
revno: 105542 [merge]
committer: Kenichi Handa <address@hidden>
branch nick: trunk
timestamp: Tue 2011-08-23 20:58:39 +0900
message:
  Fix default values of character properties.
modified:
  admin/ChangeLog
  admin/unidata/unidata-gen.el
  lisp/ChangeLog
  lisp/international/charprop.el
  lisp/international/uni-bidi.el
  lisp/international/uni-category.el
  lisp/international/uni-combining.el
  lisp/international/uni-comment.el
  lisp/international/uni-decimal.el
  lisp/international/uni-decomposition.el
  lisp/international/uni-digit.el
  lisp/international/uni-lowercase.el
  lisp/international/uni-mirrored.el
  lisp/international/uni-name.el
  lisp/international/uni-numeric.el
  lisp/international/uni-old-name.el
  lisp/international/uni-titlecase.el
  lisp/international/uni-uppercase.el
  src/ChangeLog
  src/chartab.c

=== modified file 'admin/ChangeLog'
--- a/admin/ChangeLog   2011-08-15 17:47:25 +0000
+++ b/admin/ChangeLog   2011-08-23 11:48:07 +0000
@@ -1,3 +1,17 @@
+2011-08-23  Kenichi Handa  <address@hidden>
+
+       * unidata/unidata-gen.el (unidata-prop-alist): Provide default
+       values for name, general-category, canonical-combining-class,
+       mirrored, and bidi-class.  Describe the meaning of value nil for
+       decimal-digit-value, digit-value, numeric-value, uppercase,
+       lowercase, titlecase, and mirroring.
+       (unidata-gen-table): Handle the case that default-value is a
+       list.  Set default values of characters not listed in a table.
+       (unidata-get-name): Return an empty string if a value in a
+       char-table is nil.
+       (unidata-get-decomposition): Return a list of character itself if
+       a value in a char-table is nil.
+
 2011-08-15  Eli Zaretskii  <address@hidden>
 
        * unidata/bidimirror.awk: File removed.

=== modified file 'admin/unidata/unidata-gen.el'
--- a/admin/unidata/unidata-gen.el      2011-07-06 22:43:48 +0000
+++ b/admin/unidata/unidata-gen.el      2011-08-23 11:48:07 +0000
@@ -146,7 +146,7 @@
     (setq unidata-list (cdr table))))
 
 ;; Alist of this form:
-;;   (PROP INDEX GENERATOR FILENAME DOCSTRING DESCRIBER VAL-LIST)
+;;   (PROP INDEX GENERATOR FILENAME DOCSTRING DESCRIBER DEFAULT VAL-LIST)
 ;; PROP: character property
 ;; INDEX: index to each element of unidata-list for PROP.
 ;;   It may be a function that generates an alist of character codes
@@ -155,14 +155,20 @@
 ;; FILENAME: filename to store the char-table
 ;; DOCSTRING: docstring for the property
 ;; DESCRIBER: function to call to get a description string of property value
-;; DEFAULT: the default value of the property
+;; DEFAULT: the default value of the property.  It may have the form
+;;   (VAL0 (FROM1 TO1 VAL1) ...) which indicates that the default
+;;   value is VAL0 except for characters in the ranges specified by
+;;   FROMn and TOn (incusive).  The default value of characters
+;;   between FROMn and TOn is VALn.
 ;; VAL-LIST: list of specially ordered property values
 
 (defconst unidata-prop-alist
   '((name
      1 unidata-gen-table-name "uni-name.el"
      "Unicode character name.
-Property value is a string.")
+Property value is a string."
+     nil
+     "")
     (general-category
      2 unidata-gen-table-symbol "uni-category.el"
      "Unicode general category.
@@ -170,7 +176,7 @@
   Lu, Ll, Lt, Lm, Lo, Mn, Mc, Me, Nd, Nl, No, Pc, Pd, Ps, Pe, Pi, Pf, Po,
   Sm, Sc, Sk, So, Zs, Zl, Zp, Cc, Cf, Cs, Co, Cn"
      unidata-describe-general-category
-     nil
+     Cn
      ;; The order of elements must be in sync with unicode_category_t
      ;; in src/character.h.
      (Lu Ll Lt Lm Lo Mn Mc Me Nd Nl No Pc Pd Ps Pe Pi Pf Po
@@ -179,7 +185,8 @@
      3 unidata-gen-table-integer "uni-combining.el"
      "Unicode canonical combining class.
 Property value is an integer."
-     unidata-describe-canonical-combining-class)
+     unidata-describe-canonical-combining-class
+     0)
     (bidi-class
      4 unidata-gen-table-symbol "uni-bidi.el"
      "Unicode bidi class.
@@ -187,7 +194,8 @@
   L, LRE, LRO, R, AL, RLE, RLO, PDF, EN, ES, ET,
   AN, CS, NSM, BN, B, S, WS, ON"
      unidata-describe-bidi-class
-     L
+     (L (#x0600 #x06FF AL) (#x0590 #x05FF R) (#x07C0 #x08FF R)
+       (#xFB1D #xFB4F R) (#x10800 #x10FFF R) (#x1E800 #x1EFFF R))
      ;; The order of elements must be in sync with bidi_type_t in
      ;; src/dispextern.h.
      (L R EN AN BN B AL LRE LRO RLE RLO PDF ES ET CS NSM S WS ON))
@@ -202,19 +210,24 @@
     (decimal-digit-value
      6 unidata-gen-table-integer "uni-decimal.el"
      "Unicode numeric value (decimal digit).
-Property value is an integer.")
+Property value is an integer 0..9, or nil.
+The value nil stands for NaN \"Numeric_Value\".")
     (digit-value
      7 unidata-gen-table-integer "uni-digit.el"
      "Unicode numeric value (digit).
-Property value is an integer.")
+Property value is an integer 0..9, or nil.
+The value nil stands for NaN \"Numeric_Value\".")
     (numeric-value
      8 unidata-gen-table-numeric "uni-numeric.el"
      "Unicode numeric value (numeric).
-Property value is an integer or a floating point.")
+Property value is an integer, a floating point, or nil.
+The value nil stands for NaN \"Numeric_Value\".")
     (mirrored
      9 unidata-gen-table-symbol "uni-mirrored.el"
      "Unicode bidi mirrored flag.
-Property value is a symbol `Y' or `N'.  See also the property `mirroring'.")
+Property value is a symbol `Y' or `N'.  See also the property `mirroring'."
+     nil
+     N)
     (old-name
      10 unidata-gen-table-name "uni-old-name.el"
      "Unicode old names as published in Unicode 1.0.
@@ -226,23 +239,30 @@
     (uppercase
      12 unidata-gen-table-character "uni-uppercase.el"
      "Unicode simple uppercase mapping.
-Property value is a character."
+Property value is a character or nil.
+The value nil means that the actual property value of a character
+is the character itself."
      string)
     (lowercase
      13 unidata-gen-table-character "uni-lowercase.el"
      "Unicode simple lowercase mapping.
-Property value is a character."
+Property value is a character or nil.
+The value nil means that the actual property value of a character
+is the character itself."
      string)
     (titlecase
      14 unidata-gen-table-character "uni-titlecase.el"
      "Unicode simple titlecase mapping.
-Property value is a character."
+Property value is a character or nil.
+The value nil means that the actual property value of a character
+is the character itself."
      string)
     (mirroring
      unidata-gen-mirroring-list unidata-gen-table-character "uni-mirrored.el"
      "Unicode bidi-mirroring characters.
-Property value is a character that has the corresponding mirroring image,
-or nil for non-mirrored character.")))
+Property value is a character that has the corresponding mirroring image or 
nil.
+The value nil means that the actual property value of a character
+is the character itself.")))
 
 ;; Functions to access the above data.
 (defsubst unidata-prop-index (prop) (nth 1 (assq prop unidata-prop-alist)))
@@ -393,9 +413,18 @@
     (while tail
       (setcar tail (cons (car tail) val-code))
       (setq tail (cdr tail) val-code (1+ val-code)))
-    (setq default-value (unidata-encode-val val-list default-value))
-    (set-char-table-range table t default-value)
-    (set-char-table-range table nil default-value)
+    (if (consp default-value)
+       (setq default-value (copy-sequence default-value))
+      (setq default-value (list default-value)))
+    (setcar default-value
+           (unidata-encode-val val-list (car default-value)))
+    (set-char-table-range table t (car default-value))
+    (set-char-table-range table nil (car default-value))
+    (dolist (elm (cdr default-value))
+      (setcar (nthcdr 2 elm)
+             (unidata-encode-val val-list (nth 2 elm)))
+      (set-char-table-range table (cons (car elm) (nth 1 elm)) (nth 2 elm)))
+
     (setq tail unidata-list)
     (while tail
       (setq elt (car tail) tail (cdr tail))
@@ -419,17 +448,27 @@
                (setq prev-range-data (cons (cons from to) val-code)))))
        (let* ((start (lsh (lsh range -7) 7))
               (limit (+ start 127))
-              str count new-val)
-         (fillarray vec 0)
+              str count new-val from to vcode)
+         (fillarray vec (car default-value))
+         (dolist (elm (cdr default-value))
+           (setq from (car elm) to (nth 1 elm))
+           (when (and (<= from limit)
+                      (or (>= from start) (>= to start)))
+             (setq from (max from start)
+                   to (min to limit)
+                   vcode (nth 2 elm))
+             (while (<= from to)
+               (aset vec (- from start) vcode)
+               (setq from (1+ from)))))
          ;; See the comment above.
          (when (and prev-range-data
                     (>= (cdr (car prev-range-data)) start))
-           (let ((from (car (car prev-range-data)))
-                 (to (cdr (car prev-range-data)))
-                 (vcode (cdr prev-range-data)))
-             (while (<= from to)
-               (aset vec (- from start) vcode)
-               (setq from (1+ from)))))
+           (setq from (car (car prev-range-data))
+                 to (cdr (car prev-range-data))
+                 vcode (cdr prev-range-data))
+           (while (<= from to)
+             (aset vec (- from start) vcode)
+             (setq from (1+ from))))
          (setq prev-range-data nil)
          (if val-code
              (aset vec (- range start) val-code))
@@ -669,7 +708,7 @@
            (aset table c name)
            (if (= c char)
                (setq val name))))
-       val)))
+       (or val ""))))
 
    ((and (integerp val) (> val 0))
     (let* ((symbol-table (aref (char-table-extra-slot table 4) 1))
@@ -695,7 +734,9 @@
            ((eq sym 'CJK\ COMPATIBILITY\ IDEOGRAPH)
             (format "%s-%04X" sym char))
            ((eq sym 'VARIATION\ SELECTOR)
-            (format "%s-%d" sym (+ (- char #xe0100) 17))))))))
+            (format "%s-%d" sym (+ (- char #xe0100) 17))))))
+
+   (t "")))
 
 ;; Store VAL as the name of CHAR in TABLE.
 
@@ -707,6 +748,9 @@
 
 (defun unidata-get-decomposition (char val table)
   (cond
+   ((not val)
+    (list char))
+
    ((consp val)
     val)
 
@@ -747,7 +791,8 @@
            (aset vec idx (nconc word-list tail-list)))
        (dotimes (i 128)
          (aset table (+ first-char i) (aref vec i)))
-       (aref vec (- char first-char)))))
+       (setq val (aref vec (- char first-char)))
+       (or val (list char)))))
 
    ;; Hangul syllable
    ((and (eq val 0) (>= char #xAC00) (<= char #xD7A3))

=== modified file 'lisp/ChangeLog'
--- a/lisp/ChangeLog    2011-08-23 09:31:54 +0000
+++ b/lisp/ChangeLog    2011-08-23 11:49:03 +0000
@@ -1,3 +1,21 @@
+2011-08-23  Kenichi Handa  <address@hidden>
+
+       * international/charprop.el:
+       * international/uni-bidi.el:
+       * international/uni-category.el:
+       * international/uni-combining.el:
+       * international/uni-comment.el:
+       * international/uni-decimal.el:
+       * international/uni-decomposition.el:
+       * international/uni-digit.el:
+       * international/uni-lowercase.el:
+       * international/uni-mirrored.el:
+       * international/uni-name.el:
+       * international/uni-numeric.el:
+       * international/uni-old-name.el:
+       * international/uni-titlecase.el:
+       * international/uni-uppercase.el: Regenerate.
+
 2011-08-23  Martin Rudalics  <address@hidden>
 
        * help.el (help-window-setup): Fix message displayed when other

=== modified file 'lisp/international/charprop.el'
--- a/lisp/international/charprop.el    2011-07-06 22:43:48 +0000
+++ b/lisp/international/charprop.el    2011-08-23 11:48:07 +0000
@@ -29,15 +29,18 @@
 ;; FILE: uni-decimal.el
 (define-char-code-property 'decimal-digit-value "uni-decimal.el"
   "Unicode numeric value (decimal digit).
-Property value is an integer.")
+Property value is an integer 0..9, or nil.
+The value nil stands for NaN \"Numeric_Value\".")
 ;; FILE: uni-digit.el
 (define-char-code-property 'digit-value "uni-digit.el"
   "Unicode numeric value (digit).
-Property value is an integer.")
+Property value is an integer 0..9, or nil.
+The value nil stands for NaN \"Numeric_Value\".")
 ;; FILE: uni-numeric.el
 (define-char-code-property 'numeric-value "uni-numeric.el"
   "Unicode numeric value (numeric).
-Property value is an integer or a floating point.")
+Property value is an integer, a floating point, or nil.
+The value nil stands for NaN \"Numeric_Value\".")
 ;; FILE: uni-mirrored.el
 (define-char-code-property 'mirrored "uni-mirrored.el"
   "Unicode bidi mirrored flag.
@@ -53,20 +56,27 @@
 ;; FILE: uni-uppercase.el
 (define-char-code-property 'uppercase "uni-uppercase.el"
   "Unicode simple uppercase mapping.
-Property value is a character.")
+Property value is a character or nil.
+The value nil means that the actual property value of a character
+is the character itself.")
 ;; FILE: uni-lowercase.el
 (define-char-code-property 'lowercase "uni-lowercase.el"
   "Unicode simple lowercase mapping.
-Property value is a character.")
+Property value is a character or nil.
+The value nil means that the actual property value of a character
+is the character itself.")
 ;; FILE: uni-titlecase.el
 (define-char-code-property 'titlecase "uni-titlecase.el"
   "Unicode simple titlecase mapping.
-Property value is a character.")
+Property value is a character or nil.
+The value nil means that the actual property value of a character
+is the character itself.")
 ;; FILE: uni-mirrored.el
 (define-char-code-property 'mirroring "uni-mirrored.el"
   "Unicode bidi-mirroring characters.
-Property value is a character that has the corresponding mirroring image,
-or nil for non-mirrored character.")
+Property value is a character that has the corresponding mirroring image or 
nil.
+The value nil means that the actual property value of a character
+is the character itself.")
 ;; Local Variables:
 ;; coding: utf-8
 ;; no-byte-compile: t

=== modified file 'lisp/international/uni-bidi.el'
Binary files a/lisp/international/uni-bidi.el   2011-07-06 22:43:48 +0000 and 
b/lisp/international/uni-bidi.el  2011-08-23 11:48:07 +0000 differ
=== modified file 'lisp/international/uni-category.el'
Binary files a/lisp/international/uni-category.el       2011-07-06 22:43:48 
+0000 and b/lisp/international/uni-category.el      2011-08-23 11:48:07 +0000 
differ
=== modified file 'lisp/international/uni-combining.el'
Binary files a/lisp/international/uni-combining.el      2011-07-06 22:43:48 
+0000 and b/lisp/international/uni-combining.el     2011-08-23 11:48:07 +0000 
differ
=== modified file 'lisp/international/uni-comment.el'
Binary files a/lisp/international/uni-comment.el        2011-07-06 22:43:48 
+0000 and b/lisp/international/uni-comment.el       2011-08-23 11:48:07 +0000 
differ
=== modified file 'lisp/international/uni-decimal.el'
Binary files a/lisp/international/uni-decimal.el        2011-07-06 22:43:48 
+0000 and b/lisp/international/uni-decimal.el       2011-08-23 11:48:07 +0000 
differ
=== modified file 'lisp/international/uni-decomposition.el'
Binary files a/lisp/international/uni-decomposition.el  2011-07-06 22:43:48 
+0000 and b/lisp/international/uni-decomposition.el 2011-08-23 11:48:07 +0000 
differ
=== modified file 'lisp/international/uni-digit.el'
Binary files a/lisp/international/uni-digit.el  2011-07-06 22:43:48 +0000 and 
b/lisp/international/uni-digit.el 2011-08-23 11:48:07 +0000 differ
=== modified file 'lisp/international/uni-lowercase.el'
Binary files a/lisp/international/uni-lowercase.el      2011-07-06 22:43:48 
+0000 and b/lisp/international/uni-lowercase.el     2011-08-23 11:48:07 +0000 
differ
=== modified file 'lisp/international/uni-mirrored.el'
Binary files a/lisp/international/uni-mirrored.el       2011-07-06 22:43:48 
+0000 and b/lisp/international/uni-mirrored.el      2011-08-23 11:48:07 +0000 
differ
=== modified file 'lisp/international/uni-name.el'
Binary files a/lisp/international/uni-name.el   2011-07-06 22:43:48 +0000 and 
b/lisp/international/uni-name.el  2011-08-23 11:48:07 +0000 differ
=== modified file 'lisp/international/uni-numeric.el'
Binary files a/lisp/international/uni-numeric.el        2011-07-06 22:43:48 
+0000 and b/lisp/international/uni-numeric.el       2011-08-23 11:48:07 +0000 
differ
=== modified file 'lisp/international/uni-old-name.el'
Binary files a/lisp/international/uni-old-name.el       2011-07-06 22:43:48 
+0000 and b/lisp/international/uni-old-name.el      2011-08-23 11:48:07 +0000 
differ
=== modified file 'lisp/international/uni-titlecase.el'
Binary files a/lisp/international/uni-titlecase.el      2011-07-06 22:43:48 
+0000 and b/lisp/international/uni-titlecase.el     2011-08-23 11:48:07 +0000 
differ
=== modified file 'lisp/international/uni-uppercase.el'
Binary files a/lisp/international/uni-uppercase.el      2011-07-06 22:43:48 
+0000 and b/lisp/international/uni-uppercase.el     2011-08-23 11:48:07 +0000 
differ
=== modified file 'src/ChangeLog'
--- a/src/ChangeLog     2011-08-23 10:35:47 +0000
+++ b/src/ChangeLog     2011-08-23 11:49:03 +0000
@@ -1,3 +1,8 @@
+2011-08-22  Kenichi Handa  <address@hidden>
+
+       * chartab.c (Fset_char_table_extra_slot): Do not inhibit setting a
+       extra slot even if the purpose is char-code-property-table.
+
 2011-08-23  Eli Zaretskii  <address@hidden>
 
        * xdisp.c (redisplay_window): When computing centering_position,

=== modified file 'src/chartab.c'
--- a/src/chartab.c     2011-08-15 17:47:25 +0000
+++ b/src/chartab.c     2011-08-23 11:48:07 +0000
@@ -589,8 +589,6 @@
   (Lisp_Object char_table, Lisp_Object n, Lisp_Object value)
 {
   CHECK_CHAR_TABLE (char_table);
-  if (EQ (XCHAR_TABLE (char_table)->purpose, Qchar_code_property_table))
-    error ("Can't change extra-slot of char-code-property-table");
   CHECK_NUMBER (n);
   if (XINT (n) < 0
       || XINT (n) >= CHAR_TABLE_EXTRA_SLOTS (XCHAR_TABLE (char_table)))