emacs-bug-tracker
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

[debbugs-tracker] bug#19859: closed (25.0.50; [PATCH] calendar/solar.el


From: GNU bug Tracking System
Subject: [debbugs-tracker] bug#19859: closed (25.0.50; [PATCH] calendar/solar.el (sunrise-sunset): Remove temp-buffer output.)
Date: Fri, 20 Feb 2015 06:26:02 +0000

Your message dated Fri, 20 Feb 2015 01:25:06 -0500
with message-id <address@hidden>
and subject line Re: bug#19859: 25.0.50; [PATCH] calendar/solar.el 
(sunrise-sunset): Remove temp-buffer output.
has caused the debbugs.gnu.org bug report #19859,
regarding 25.0.50; [PATCH] calendar/solar.el (sunrise-sunset): Remove 
temp-buffer output.
to be marked as done.

(If you believe you have received this mail in error, please contact
address@hidden)


-- 
19859: http://debbugs.gnu.org/cgi/bugreport.cgi?bug=19859
GNU Bug Tracking System
Contact address@hidden with problems
--- Begin Message --- Subject: 25.0.50; [PATCH] calendar/solar.el (sunrise-sunset): Remove temp-buffer output. Date: Fri, 13 Feb 2015 17:05:20 -0500
---
 lisp/ChangeLog         |  4 ++++
 lisp/calendar/solar.el | 16 ++--------------
 2 files changed, 6 insertions(+), 14 deletions(-)

Having a temp buffer pop up for something that isn't going to be more
than two lines (wrapped) in the echo area is pretty annoying.  It
doesn't make much sense either that the output in the temp buffer is
split over two lines, so now one has two lines of output in a new buffer
instead of just the minibuffer, and now one has to hit keys (do work) to
make the buffer go away.

diff --git a/lisp/ChangeLog b/lisp/ChangeLog
index 030d572..3a7f27d 100644
--- a/lisp/ChangeLog
+++ b/lisp/ChangeLog
@@ -1,3 +1,7 @@
+2015-02-13  Mark Oteiza  <address@hidden>
+
+       * calendar/solar.el (sunrise-sunset): Remove temp-buffer output.
+
 2015-02-13  Artur Malabarba  <address@hidden>
 
        * emacs-lisp/package.el (package--compatibility-table): New var.
diff --git a/lisp/calendar/solar.el b/lisp/calendar/solar.el
index 46cd8d3..9e6bc5e 100644
--- a/lisp/calendar/solar.el
+++ b/lisp/calendar/solar.el
@@ -847,20 +847,8 @@ This function is suitable for execution in an init file."
          (date (if (< arg 4) (calendar-current-date) (calendar-read-date)))
          (date-string (calendar-date-string date t))
          (time-string (solar-sunrise-sunset-string date))
-         (msg (format "%s: %s" date-string time-string))
-         (one-window (one-window-p t)))
-    (if (<= (length msg) (frame-width))
-        (message "%s" msg)
-      (with-output-to-temp-buffer "*temp*"
-        (princ (concat date-string "\n" time-string)))
-      (message "%s"
-               (substitute-command-keys
-                (if one-window
-                    (if pop-up-windows
-                        "Type \\[delete-other-windows] to remove temp window."
-                      "Type \\[switch-to-buffer] RET to remove temp window.")
-                  "Type \\[switch-to-buffer-other-window] RET to restore old \
-contents of temp window."))))))
+         (msg (format "%s: %s" date-string time-string)))
+    (message "%s" msg)))
 
 ;;;###cal-autoload
 (defun calendar-sunrise-sunset (&optional event)
-- 
2.3.0




--- End Message ---
--- Begin Message --- Subject: Re: bug#19859: 25.0.50; [PATCH] calendar/solar.el (sunrise-sunset): Remove temp-buffer output. Date: Fri, 20 Feb 2015 01:25:06 -0500 User-agent: Gnus (www.gnus.org), GNU Emacs (www.gnu.org/software/emacs/)
Version: 25.1

Fair enough; I applied something along those lines, thanks.


--- End Message ---

reply via email to

[Prev in Thread] Current Thread [Next in Thread]