lisp/calculator.el (calculator-last-input): drop 'ascii-character property

lookup.
etc/NEWS: calculator.el user-visible changes.

diff --git a/etc/ChangeLog b/etc/ChangeLog
index 4419f4a0e660226275ee55a90ef8d1d0bf22d24c..fbeaed956a45f4d486e970322c2352bcedb14ba8 100644 (file)
--- a/etc/ChangeLog
+++ b/etc/ChangeLog
@@ -1,3 +1,7 @@
+2014-06-24  Eli Barzilay  <eli@barzilay.org>
+
+       * NEWS: calculator.el user-visible changes.
+
 2014-06-15  Michael Albinus  <michael.albinus@gmx.de>
 
        * NEWS: New Tramp method "nc".

diff --git a/etc/NEWS b/etc/NEWS
index 27e9cfa0e3c4d10528749787c8dc2429cdcac853..2114574b2d131e551b40df4cdfc22c6edd91d66c 100644 (file)
--- a/etc/NEWS
+++ b/etc/NEWS
@@ -113,6 +113,10 @@ protocols as well as for "telnet" and "ftp" are passed to Tramp.
 
 *** New connection method "nc", which allows to access dumb busyboxes.
 
+** Calculator: decimal display mode uses "," groups, so it's more
+fitting for use in money calculations; factorial works with
+non-integer inputs.
+
 ** Obsolete packages
 
 ---

diff --git a/lisp/ChangeLog b/lisp/ChangeLog
index 3ed3d16e69fa5c1f79b80f32b5835402d9040480..1ce15a3c8b651f4248b12e7a550eeb8128271c6e 100644 (file)
--- a/lisp/ChangeLog
+++ b/lisp/ChangeLog
@@ -1,3 +1,8 @@
+2014-06-24  Eli Barzilay  <eli@barzilay.org>
+
+       * calculator.el (calculator-last-input): drop 'ascii-character property
+       lookup.
+
 2014-06-24  Leo Liu  <sdl.web@gmail.com>
 
        * align.el (align-adjust-col-for-rule): Unbreak due to defaulting

diff --git a/lisp/calculator.el b/lisp/calculator.el
index 9ffa6b1a64bf7535ead83bedf5a6625a824251bf..ad7a7f4c92a3beeefcf48e6cddd37e00a6df5a10 100644 (file)
--- a/lisp/calculator.el
+++ b/lisp/calculator.el
@@ -1217,13 +1217,11 @@ Use KEYS if given, otherwise use `this-command-keys'."
       inp
       ;; Translates kp-x to x and [tries to] create a string to lookup
       ;; operators; assume all symbols are translatable via
-      ;; `function-key-map' or with an 'ascii-character property.  This
-      ;; is needed because we have key bindings for kp-* (which might be
-      ;; the wrong thing to do) so they don't get translated in
-      ;; `this-command-keys'.
+      ;; `function-key-map'.  This is needed because we have key
+      ;; bindings for kp-* (which might be the wrong thing to do) so
+      ;; they don't get translated in `this-command-keys'.
       (concat (mapcar (lambda (k)
-                        (if (numberp k) k (or (get k 'ascii-character)
-                                              (error "??bad key??"))))
+                        (if (numberp k) k (error "??bad key?? (%S)" k)))
                       (or (lookup-key function-key-map inp) inp))))))
 
 (defun calculator-clear-fragile (&optional op)