bug#5674: 23.1; calculator moves point

bug#5674: 23.1; calculator moves point

[Sebastian Rose]

[-- Attachment #1: Type: text/plain, Size: 591 bytes --]


Please write in English if possible, because the Emacs maintainers
usually do not have translators to read other languages for them.

Your bug report will be posted to the bug-gnu-emacs@gnu.org mailing list,
and to the gnu.emacs.bug news group.

Please describe exactly what actions triggered the bug
and the precise symptoms of the bug:


In the current development head only:

`M-x calculator' moves point to some arbitrary point in the buffer. You
may test this in *scratch* buffer after startup (point will be moved to
the end of the buffer in this case).

This simple patch fixes it:


[-- Warning: decoded text below may be mangled, UTF-8 assumed --]
[-- Attachment #2: Type: text/x-verbatim, Size: 485 bytes --]

--- lisp/calculator.el	2010-01-21 13:34:34.000000000 +0100
+++ /home/sebastian/Desktop/calculator.el	2010-03-01 20:50:12.000000000 +0100
@@ -724,7 +724,7 @@
     (progn
       (cond
         ((not (get-buffer-window calculator-buffer))
-         (let ((split-window-keep-point nil)
+         (let ((split-window-keep-point t)
                (window-min-height 2))
            ;; maybe leave two lines for our window because of the normal
            ;; `raised' modeline in Emacs 21


[-- Attachment #3: Type: text/plain, Size: 391 bytes --]


-- 
~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
Sebastian  Rose      Fachinformatiker / Anwendungsentwicklung
Viktoriastr. 22      Entwicklung von Anwendungen mit freien Werkzeugen
30451  Hannover      und Bibliotheken.

0173  83 93 417      sebastian_rose@gmx.de         s.rose@emma-stil.de
~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~

bug#5674: 23.1; calculator moves point

[Chong Yidong]

> `M-x calculator' moves point to some arbitrary point in the buffer. You
> may test this in *scratch* buffer after startup (point will be moved to
> the end of the buffer in this case).
>
> This simple patch fixes it:

Better not to bind split-window-keep-point at all, I think (not sure why
the code did it).  I've checked in just such a change.  Thanks for the
bug report.