* bug#3862: 23.1.50; error calling `documentation' on calendar-mode @ 2009-07-15 23:50 ` Kevin Ryde 2009-07-16 1:40 ` Stefan Monnier 2009-07-19 0:45 ` bug#3862: marked as done (23.1.50; error calling `documentation' on calendar-mode) Emacs bug Tracking System 0 siblings, 2 replies; 6+ messages in thread From: Kevin Ryde @ 2009-07-15 23:50 UTC (permalink / raw) To: emacs-pretest-bug In a recent cvs starting "emacs -Q" and evaluating (progn (require 'calendar) (documentation 'calendar-mode)) gets the error and backtrace below. For what it's worth it works fine from the actual calendar buffer, but I was trying in some program code where that wasn't the case. I hoped I could generally get the docs for any function from anywhere. Also incidentally I tried a condition-case around it like (condition-case nil (documentation 'calendar-mode) (error nil)) and was surprised it still went to the debugger. But maybe that's a separate problem. Debugger entered--Lisp error: (void-variable displayed-month) calendar-cursor-to-date(t 24) cal-menu-set-date-title(("--" ["Holidays" calendar-cursor-holidays] ["Mark date" calendar-set-mark] ["Sunrise/sunset" calendar-sunrise-sunset] ["Other calendars" calendar-print-other-dates] ("Prepare LaTeX buffer" ["Daily (1 page)" cal-tex-cursor-day] ["Weekly (1 page)" cal-tex-cursor-week] ["Weekly (2 pages)" cal-tex-cursor-week2] ["Weekly (other style; 1 page)" cal-tex-cursor-week-iso] ["Weekly (yet another style; 1 page)" cal-tex-cursor-week-monday] ["Monthly" cal-tex-cursor-month] ["Monthly (landscape)" cal-tex-cursor-month-landscape] ["Yearly" cal-tex-cursor-year] ["Yearly (landscape)" cal-tex-cursor-year-landscape] ("Filofax styles" ["Filofax Daily (one-day-per-page)" cal-tex-cursor-filofax-daily] ["Filofax Weekly (2-weeks-at-a-glance)" cal-tex-cursor-filofax-2week] ["Filofax Weekly (week-at-a-glance)" cal-tex-cursor-filofax-week] ["Filofax Yearly" cal-tex-cursor-filofax-year])) ("Write HTML calendar" ["For selected month" cal-html-cursor-month] ["For selected year" cal-html-cursor-year]) ["Diary entries" calendar-mouse-view-diary-entries :keys "d"] ["Insert diary entry" diary-insert-entry] ["Other diary file entries" calendar-mouse-view-other-diary-entries :keys "D"])) (easy-menu-filter-return (cal-menu-set-date-title menu) "cal-menu-mouse2") (lambda (menu) (easy-menu-filter-return (cal-menu-set-date-title menu) "cal-menu-mouse2"))(("--" ["Holidays" calendar-cursor-holidays] ["Mark date" calendar-set-mark] ["Sunrise/sunset" calendar-sunrise-sunset] ["Other calendars" calendar-print-other-dates] ("Prepare LaTeX buffer" ["Daily (1 page)" cal-tex-cursor-day] ["Weekly (1 page)" cal-tex-cursor-week] ["Weekly (2 pages)" cal-tex-cursor-week2] ["Weekly (other style; 1 page)" cal-tex-cursor-week-iso] ["Weekly (yet another style; 1 page)" cal-tex-cursor-week-monday] ["Monthly" cal-tex-cursor-month] ["Monthly (landscape)" cal-tex-cursor-month-landscape] ["Yearly" cal-tex-cursor-year] ["Yearly (landscape)" cal-tex-cursor-year-landscape] ("Filofax styles" ["Filofax Daily (one-day-per-page)" cal-tex-cursor-filofax-daily] ["Filofax Weekly (2-weeks-at-a-glance)" cal-tex-cursor-filofax-2week] ["Filofax Weekly (week-at-a-glance)" cal-tex-cursor-filofax-week] ["Filofax Yearly" cal-tex-cursor-filofax-year])) ("Write HTML calendar" ["For selected month" cal-html-cursor-month] ["For selected year" cal-html-cursor-year]) ["Diary entries" calendar-mouse-view-diary-entries :keys "d"] ["Insert diary entry" diary-insert-entry] ["Other diary file entries" calendar-mouse-view-other-diary-entries :keys "D"])) documentation(calendar-mode) (progn (require (quote calendar)) (documentation (quote calendar-mode))) eval-region(1 63 t (lambda (ignore) (goto-char 63) (quote (progn ... ...)))) ; Reading at buffer position 8 apply(eval-region (1 63 t (lambda (ignore) (goto-char 63) (quote ...)))) eval-defun-2() eval-defun(nil) call-interactively(eval-defun nil nil) In GNU Emacs 23.1.50.1 (i586-pc-linux-gnu, GTK+ Version 2.16.4) of 2009-07-12 on blah.blah configured using `configure 'CFLAGS=-O -g' '--prefix=/down/emacs/b/inst' '--with-x-toolkit=gtk'' Important settings: value of $LC_ALL: nil value of $LC_COLLATE: nil value of $LC_CTYPE: nil value of $LC_MESSAGES: nil value of $LC_MONETARY: nil value of $LC_NUMERIC: nil value of $LC_TIME: nil value of $LANG: en_AU value of $XMODIFIERS: nil locale-coding-system: iso-latin-1-unix default-enable-multibyte-characters: t ^ permalink raw reply [flat|nested] 6+ messages in thread
* bug#3862: 23.1.50; error calling `documentation' on calendar-mode 2009-07-15 23:50 ` bug#3862: 23.1.50; error calling `documentation' on calendar-mode Kevin Ryde @ 2009-07-16 1:40 ` Stefan Monnier 2009-07-17 1:02 ` Glenn Morris 2009-07-19 0:45 ` bug#3862: marked as done (23.1.50; error calling `documentation' on calendar-mode) Emacs bug Tracking System 1 sibling, 1 reply; 6+ messages in thread From: Stefan Monnier @ 2009-07-16 1:40 UTC (permalink / raw) To: Kevin Ryde; +Cc: 3862 > (progn > (require 'calendar) > (documentation 'calendar-mode)) > gets the error and backtrace below. The docstring gets passed through substitute-command-keys which can run arbitrary code via menu-item :filters (as well as via autoloaded keymaps, actually). > For what it's worth it works fine from the actual calendar buffer, but I > was trying in some program code where that wasn't the case. I hoped I > could generally get the docs for any function from anywhere. In the present case, there's a subtle bug in the :filter of the calendar-mode-map (more specially in the menu-bar part of the map). > Also incidentally I tried a condition-case around it like > (condition-case nil > (documentation 'calendar-mode) > (error nil)) > and was surprised it still went to the debugger. But maybe that's a > separate problem. Indeed it's a separate problem: when the filter is run, it is presumed it is "at top level" and so the error gets turned into a backtrace without paying attention to surrounding condition-cases. It's clearly a bug, but it might be delicate to fix without introducing risks of nasty crashes. Stefan ^ permalink raw reply [flat|nested] 6+ messages in thread
* bug#3862: 23.1.50; error calling `documentation' on calendar-mode 2009-07-16 1:40 ` Stefan Monnier @ 2009-07-17 1:02 ` Glenn Morris 2009-07-17 5:12 ` Glenn Morris 0 siblings, 1 reply; 6+ messages in thread From: Glenn Morris @ 2009-07-17 1:02 UTC (permalink / raw) To: Stefan Monnier; +Cc: Kevin Ryde, 3862 Stefan Monnier wrote: > In the present case, there's a subtle bug in the :filter of the > calendar-mode-map (more specially in the menu-bar part of the map). Could you explain what the bug is? The error is coming from cal-menu-set-date-title in a mouse-binding, which calls calendar-cursor-to-date. This has always assumed it is called from a calendar buffer, which seems reasonable, and gives an error if it is not. Also, why does C-h f calendar-mode work fine? On a related note, I notice that the Emacs 22 behaviour, where it displays an explicit error message "Not on a date" if called in the calendar with point not a date, has been lost - now it is just silent. I would like to get this back, but I can't figure out the Easymenu-Way to do it. Anyway, the following hack will suppress the original issue. *** calendar.el 8 Jul 2009 02:48:13 -0000 1.283 --- calendar.el 17 Jul 2009 00:53:33 -0000 *************** *** 1811,1816 **** --- 1811,1817 ---- (with-current-buffer (if event (window-buffer (posn-window (event-start event))) (current-buffer)) + (or (boundp 'displayed-month) (error "Not on a date!")) (save-excursion (and event (setq event (event-start event)) (goto-char (posn-point event))) ^ permalink raw reply [flat|nested] 6+ messages in thread
* bug#3862: 23.1.50; error calling `documentation' on calendar-mode 2009-07-17 1:02 ` Glenn Morris @ 2009-07-17 5:12 ` Glenn Morris 2009-07-17 15:58 ` Stefan Monnier 0 siblings, 1 reply; 6+ messages in thread From: Glenn Morris @ 2009-07-17 5:12 UTC (permalink / raw) To: 3862; +Cc: Kevin Ryde Perhaps something like this is better: *** cal-menu.el.~1.114.~ 2009-01-07 20:13:09.000000000 -0800 --- cal-menu.el 2009-07-16 22:08:54.000000000 -0700 *************** *** 202,210 **** (defun cal-menu-set-date-title (menu) "Convert date of last event to title suitable for MENU." ! (easy-menu-filter-return ! menu (calendar-date-string (calendar-cursor-to-date t last-input-event) ! t nil))) (easy-menu-define cal-menu-context-mouse-menu nil "Pop up menu for Mouse-2 for selected date in the calendar window." --- 202,212 ---- (defun cal-menu-set-date-title (menu) "Convert date of last event to title suitable for MENU." ! (let ((date (ignore-errors (calendar-cursor-to-date nil last-input-event)))) ! (if date ! (easy-menu-filter-return menu (calendar-date-string date t nil)) ! (message "Not on a date!") ! nil))) (easy-menu-define cal-menu-context-mouse-menu nil "Pop up menu for Mouse-2 for selected date in the calendar window." ^ permalink raw reply [flat|nested] 6+ messages in thread
* bug#3862: 23.1.50; error calling `documentation' on calendar-mode 2009-07-17 5:12 ` Glenn Morris @ 2009-07-17 15:58 ` Stefan Monnier 0 siblings, 0 replies; 6+ messages in thread From: Stefan Monnier @ 2009-07-17 15:58 UTC (permalink / raw) To: Glenn Morris; +Cc: Kevin Ryde, 3862 > Perhaps something like this is better: That seems OK, tho I'd rather return the menu in either case and use some default string (e.g. "today") for the date when calendar-cursor-to-date signals an error. Stefan ^ permalink raw reply [flat|nested] 6+ messages in thread
* bug#3862: marked as done (23.1.50; error calling `documentation' on calendar-mode) 2009-07-15 23:50 ` bug#3862: 23.1.50; error calling `documentation' on calendar-mode Kevin Ryde 2009-07-16 1:40 ` Stefan Monnier @ 2009-07-19 0:45 ` Emacs bug Tracking System 1 sibling, 0 replies; 6+ messages in thread From: Emacs bug Tracking System @ 2009-07-19 0:45 UTC (permalink / raw) To: Glenn Morris [-- Attachment #1: Type: text/plain, Size: 912 bytes --] Your message dated Sat, 18 Jul 2009 20:37:04 -0400 with message-id <6dfxct1pan.fsf@fencepost.gnu.org> and subject line Re: bug#3862: 23.1.50; error calling `documentation' on calendar-mode has caused the Emacs bug report #3862, regarding 23.1.50; error calling `documentation' on calendar-mode to be marked as done. This means that you claim that the problem has been dealt with. If this is not the case it is now your responsibility to reopen the bug report if necessary, and/or fix the problem forthwith. (NB: If you are a system administrator and have no idea what this message is talking about, this may indicate a serious mail system misconfiguration somewhere. Please contact owner@emacsbugs.donarmstrong.com immediately.) -- 3862: http://emacsbugs.donarmstrong.com/cgi-bin/bugreport.cgi?bug=3862 Emacs Bug Tracking System Contact owner@emacsbugs.donarmstrong.com with problems [-- Attachment #2: Type: message/rfc822, Size: 6604 bytes --] From: Kevin Ryde <user42@zip.com.au> To: emacs-pretest-bug@gnu.org Subject: 23.1.50; error calling `documentation' on calendar-mode Date: Thu, 16 Jul 2009 09:50:38 +1000 Message-ID: <87tz1da4kx.fsf@blah.blah> In a recent cvs starting "emacs -Q" and evaluating (progn (require 'calendar) (documentation 'calendar-mode)) gets the error and backtrace below. For what it's worth it works fine from the actual calendar buffer, but I was trying in some program code where that wasn't the case. I hoped I could generally get the docs for any function from anywhere. Also incidentally I tried a condition-case around it like (condition-case nil (documentation 'calendar-mode) (error nil)) and was surprised it still went to the debugger. But maybe that's a separate problem. Debugger entered--Lisp error: (void-variable displayed-month) calendar-cursor-to-date(t 24) cal-menu-set-date-title(("--" ["Holidays" calendar-cursor-holidays] ["Mark date" calendar-set-mark] ["Sunrise/sunset" calendar-sunrise-sunset] ["Other calendars" calendar-print-other-dates] ("Prepare LaTeX buffer" ["Daily (1 page)" cal-tex-cursor-day] ["Weekly (1 page)" cal-tex-cursor-week] ["Weekly (2 pages)" cal-tex-cursor-week2] ["Weekly (other style; 1 page)" cal-tex-cursor-week-iso] ["Weekly (yet another style; 1 page)" cal-tex-cursor-week-monday] ["Monthly" cal-tex-cursor-month] ["Monthly (landscape)" cal-tex-cursor-month-landscape] ["Yearly" cal-tex-cursor-year] ["Yearly (landscape)" cal-tex-cursor-year-landscape] ("Filofax styles" ["Filofax Daily (one-day-per-page)" cal-tex-cursor-filofax-daily] ["Filofax Weekly (2-weeks-at-a-glance)" cal-tex-cursor-filofax-2week] ["Filofax Weekly (week-at-a-glance)" cal-tex-cursor-filofax-week] ["Filofax Yearly" cal-tex-cursor-filofax-year])) ("Write HTML calendar" ["For selected month" cal-html-cursor-month] ["For selected year" cal-html-cursor-year]) ["Diary entries" calendar-mouse-view-diary-entries :keys "d"] ["Insert diary entry" diary-insert-entry] ["Other diary file entries" calendar-mouse-view-other-diary-entries :keys "D"])) (easy-menu-filter-return (cal-menu-set-date-title menu) "cal-menu-mouse2") (lambda (menu) (easy-menu-filter-return (cal-menu-set-date-title menu) "cal-menu-mouse2"))(("--" ["Holidays" calendar-cursor-holidays] ["Mark date" calendar-set-mark] ["Sunrise/sunset" calendar-sunrise-sunset] ["Other calendars" calendar-print-other-dates] ("Prepare LaTeX buffer" ["Daily (1 page)" cal-tex-cursor-day] ["Weekly (1 page)" cal-tex-cursor-week] ["Weekly (2 pages)" cal-tex-cursor-week2] ["Weekly (other style; 1 page)" cal-tex-cursor-week-iso] ["Weekly (yet another style; 1 page)" cal-tex-cursor-week-monday] ["Monthly" cal-tex-cursor-month] ["Monthly (landscape)" cal-tex-cursor-month-landscape] ["Yearly" cal-tex-cursor-year] ["Yearly (landscape)" cal-tex-cursor-year-landscape] ("Filofax styles" ["Filofax Daily (one-day-per-page)" cal-tex-cursor-filofax-daily] ["Filofax Weekly (2-weeks-at-a-glance)" cal-tex-cursor-filofax-2week] ["Filofax Weekly (week-at-a-glance)" cal-tex-cursor-filofax-week] ["Filofax Yearly" cal-tex-cursor-filofax-year])) ("Write HTML calendar" ["For selected month" cal-html-cursor-month] ["For selected year" cal-html-cursor-year]) ["Diary entries" calendar-mouse-view-diary-entries :keys "d"] ["Insert diary entry" diary-insert-entry] ["Other diary file entries" calendar-mouse-view-other-diary-entries :keys "D"])) documentation(calendar-mode) (progn (require (quote calendar)) (documentation (quote calendar-mode))) eval-region(1 63 t (lambda (ignore) (goto-char 63) (quote (progn ... ...)))) ; Reading at buffer position 8 apply(eval-region (1 63 t (lambda (ignore) (goto-char 63) (quote ...)))) eval-defun-2() eval-defun(nil) call-interactively(eval-defun nil nil) In GNU Emacs 23.1.50.1 (i586-pc-linux-gnu, GTK+ Version 2.16.4) of 2009-07-12 on blah.blah configured using `configure 'CFLAGS=-O -g' '--prefix=/down/emacs/b/inst' '--with-x-toolkit=gtk'' Important settings: value of $LC_ALL: nil value of $LC_COLLATE: nil value of $LC_CTYPE: nil value of $LC_MESSAGES: nil value of $LC_MONETARY: nil value of $LC_NUMERIC: nil value of $LC_TIME: nil value of $LANG: en_AU value of $XMODIFIERS: nil locale-coding-system: iso-latin-1-unix default-enable-multibyte-characters: t [-- Attachment #3: Type: message/rfc822, Size: 1760 bytes --] From: Glenn Morris <rgm@gnu.org> To: 3862-done@emacsbugs.donarmstrong.com Subject: Re: bug#3862: 23.1.50; error calling `documentation' on calendar-mode Date: Sat, 18 Jul 2009 20:37:04 -0400 Message-ID: <6dfxct1pan.fsf@fencepost.gnu.org> * calendar/cal-menu.el (cal-menu-set-date-title): Handle calls from non-calendar buffers (Bug#3862). ^ permalink raw reply [flat|nested] 6+ messages in thread
end of thread, other threads:[~2009-07-19 0:45 UTC | newest]
Thread overview: 6+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
[not found] <6dfxct1pan.fsf@fencepost.gnu.org>
2009-07-15 23:50 ` bug#3862: 23.1.50; error calling `documentation' on calendar-mode Kevin Ryde
2009-07-16 1:40 ` Stefan Monnier
2009-07-17 1:02 ` Glenn Morris
2009-07-17 5:12 ` Glenn Morris
2009-07-17 15:58 ` Stefan Monnier
2009-07-19 0:45 ` bug#3862: marked as done (23.1.50; error calling `documentation' on calendar-mode) Emacs bug Tracking System
Code repositories for project(s) associated with this public inbox https://git.savannah.gnu.org/cgit/emacs.git This is a public inbox, see mirroring instructions for how to clone and mirror all data and code used for this inbox; as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).