* bug#1551: 23.0.60; declare-function: redundant `nil' arg
@ 2008-12-12 14:23 ` Stephen Berman
2008-12-12 19:15 ` bug#1551: marked as done (23.0.60; declare-function: redundant `nil' arg) Emacs bug Tracking System
0 siblings, 1 reply; 3+ messages in thread
From: Stephen Berman @ 2008-12-12 14:23 UTC (permalink / raw
To: emacs-pretest-bug
[-- Attachment #1: Type: text/plain, Size: 687 bytes --]
In GNU Emacs 23.0.60.23 (i686-pc-linux-gnu, GTK+ Version 2.12.9) of
2008-12-12 on escher
According to the argument-list of declare-function, only its first two
arguments are obligatory. But by rgrepping emacs/lisp for
"(declare-function" I found four invocations with `nil' as the third
argument (there are many more invocations with only two arguments):
./calendar/todo-mode.el:914:(declare-function calendar-current-date "calendar" nil)
./progmodes/fortran.el:917:(declare-function gud-find-c-expr "gud.el" nil)
./smerge-mode.el:1079:(declare-function ediff-cleanup-mess "ediff-util" nil)
./subr.el:2251:(declare-function w32-shell-dos-semantics "w32-fns" nil)
Patches attached.
[-- Warning: decoded text below may be mangled, UTF-8 assumed --]
[-- Attachment #2: smerge-mode.el patch --]
[-- Type: text/x-patch, Size: 686 bytes --]
*** /home/steve/cvsroot/emacs/lisp/smerge-mode.el.~1.74.~ 2008-11-04 21:09:18.000000000 +0100
--- /home/steve/cvsroot/emacs/lisp/smerge-mode.el 2008-12-12 15:03:30.000000000 +0100
***************
*** 1076,1082 ****
(defvar ediff-buffer-C)
(defvar ediff-ancestor-buffer)
(defvar ediff-quit-hook)
! (declare-function ediff-cleanup-mess "ediff-util" nil)
;;;###autoload
(defun smerge-ediff (&optional name-mine name-other name-base)
--- 1076,1082 ----
(defvar ediff-buffer-C)
(defvar ediff-ancestor-buffer)
(defvar ediff-quit-hook)
! (declare-function ediff-cleanup-mess "ediff-util")
;;;###autoload
(defun smerge-ediff (&optional name-mine name-other name-base)
[-- Warning: decoded text below may be mangled, UTF-8 assumed --]
[-- Attachment #3: subr.el patch --]
[-- Type: text/x-patch, Size: 753 bytes --]
*** /home/steve/cvsroot/emacs/lisp/subr.el.~1.623.~ 2008-11-20 13:49:52.000000000 +0100
--- /home/steve/cvsroot/emacs/lisp/subr.el 2008-12-12 15:04:49.000000000 +0100
***************
*** 2248,2254 ****
(play-sound-internal sound)
(error "This Emacs binary lacks sound support")))
! (declare-function w32-shell-dos-semantics "w32-fns" nil)
(defun shell-quote-argument (argument)
"Quote an argument for passing as argument to an inferior shell."
--- 2248,2254 ----
(play-sound-internal sound)
(error "This Emacs binary lacks sound support")))
! (declare-function w32-shell-dos-semantics "w32-fns")
(defun shell-quote-argument (argument)
"Quote an argument for passing as argument to an inferior shell."
[-- Warning: decoded text below may be mangled, UTF-8 assumed --]
[-- Attachment #4: todo-mode.el patch --]
[-- Type: text/x-patch, Size: 694 bytes --]
*** /home/steve/cvsroot/emacs/lisp/calendar/todo-mode.el.~1.71.~ 2008-10-25 20:48:16.000000000 +0200
--- /home/steve/cvsroot/emacs/lisp/calendar/todo-mode.el 2008-12-12 14:57:25.000000000 +0100
***************
*** 911,917 ****
(defvar entry)
;; t-c should be used from diary code, which requires calendar.
! (declare-function calendar-current-date "calendar" nil)
;; Read about this function in the setup instructions above!
;;;###autoload
--- 911,917 ----
(defvar entry)
;; t-c should be used from diary code, which requires calendar.
! (declare-function calendar-current-date "calendar")
;; Read about this function in the setup instructions above!
;;;###autoload
[-- Warning: decoded text below may be mangled, UTF-8 assumed --]
[-- Attachment #5: fortran.el patch --]
[-- Type: text/x-patch, Size: 745 bytes --]
*** /home/steve/cvsroot/emacs/lisp/progmodes/fortran.el.~1.151.~ 2008-11-20 13:49:53.000000000 +0100
--- /home/steve/cvsroot/emacs/lisp/progmodes/fortran.el 2008-12-12 14:59:26.000000000 +0100
***************
*** 914,920 ****
"Fortran mode adds this to `hack-local-variables-hook'."
(fortran-line-length fortran-line-length))
! (declare-function gud-find-c-expr "gud.el" nil)
(defun fortran-gud-find-expr ()
;; Consider \n as punctuation (end of expression).
--- 914,920 ----
"Fortran mode adds this to `hack-local-variables-hook'."
(fortran-line-length fortran-line-length))
! (declare-function gud-find-c-expr "gud.el")
(defun fortran-gud-find-expr ()
;; Consider \n as punctuation (end of expression).
^ permalink raw reply [flat|nested] 3+ messages in thread
* Processed: Re: bug#1551: 23.0.60; declare-function: redundant `nil' arg
[not found] <078wqlky82.fsf@fencepost.gnu.org>
2008-12-12 14:23 ` bug#1551: 23.0.60; declare-function: redundant `nil' arg Stephen Berman
@ 2008-12-12 19:15 ` Emacs bug Tracking System
1 sibling, 0 replies; 3+ messages in thread
From: Emacs bug Tracking System @ 2008-12-12 19:15 UTC (permalink / raw
To: Glenn Morris; +Cc: Emacs Bugs
Processing commands for control@emacsbugs.donarmstrong.com:
> tags 1551 notabug
bug#1551: 23.0.60; declare-function: redundant `nil' arg
There were no tags set.
Tags added: notabug
> stop
Stopping processing here.
Please contact me if you need assistance.
Don Armstrong
(administrator, Emacs bugs database)
^ permalink raw reply [flat|nested] 3+ messages in thread
* bug#1551: marked as done (23.0.60; declare-function: redundant `nil' arg)
2008-12-12 14:23 ` bug#1551: 23.0.60; declare-function: redundant `nil' arg Stephen Berman
@ 2008-12-12 19:15 ` Emacs bug Tracking System
0 siblings, 0 replies; 3+ messages in thread
From: Emacs bug Tracking System @ 2008-12-12 19:15 UTC (permalink / raw
To: Glenn Morris
[-- Attachment #1: Type: text/plain, Size: 870 bytes --]
Your message dated Fri, 12 Dec 2008 14:08:45 -0500
with message-id <078wqlky82.fsf@fencepost.gnu.org>
and subject line Re: bug#1551: 23.0.60; declare-function: redundant `nil' arg
has caused the Emacs bug report #1551,
regarding 23.0.60; declare-function: redundant `nil' arg
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 don@donarmstrong.com
immediately.)
--
1551: http://emacsbugs.donarmstrong.com/cgi-bin/bugreport.cgi?bug=1551
Emacs Bug Tracking System
Contact don@donarmstrong.com with problems
[-- Attachment #2: Type: message/rfc822, Size: 6750 bytes --]
[-- Attachment #2.1.1: Type: text/plain, Size: 687 bytes --]
In GNU Emacs 23.0.60.23 (i686-pc-linux-gnu, GTK+ Version 2.12.9) of
2008-12-12 on escher
According to the argument-list of declare-function, only its first two
arguments are obligatory. But by rgrepping emacs/lisp for
"(declare-function" I found four invocations with `nil' as the third
argument (there are many more invocations with only two arguments):
./calendar/todo-mode.el:914:(declare-function calendar-current-date "calendar" nil)
./progmodes/fortran.el:917:(declare-function gud-find-c-expr "gud.el" nil)
./smerge-mode.el:1079:(declare-function ediff-cleanup-mess "ediff-util" nil)
./subr.el:2251:(declare-function w32-shell-dos-semantics "w32-fns" nil)
Patches attached.
[-- Warning: decoded text below may be mangled, UTF-8 assumed --]
[-- Attachment #2.1.2: smerge-mode.el patch --]
[-- Type: text/x-patch, Size: 686 bytes --]
*** /home/steve/cvsroot/emacs/lisp/smerge-mode.el.~1.74.~ 2008-11-04 21:09:18.000000000 +0100
--- /home/steve/cvsroot/emacs/lisp/smerge-mode.el 2008-12-12 15:03:30.000000000 +0100
***************
*** 1076,1082 ****
(defvar ediff-buffer-C)
(defvar ediff-ancestor-buffer)
(defvar ediff-quit-hook)
! (declare-function ediff-cleanup-mess "ediff-util" nil)
;;;###autoload
(defun smerge-ediff (&optional name-mine name-other name-base)
--- 1076,1082 ----
(defvar ediff-buffer-C)
(defvar ediff-ancestor-buffer)
(defvar ediff-quit-hook)
! (declare-function ediff-cleanup-mess "ediff-util")
;;;###autoload
(defun smerge-ediff (&optional name-mine name-other name-base)
[-- Warning: decoded text below may be mangled, UTF-8 assumed --]
[-- Attachment #2.1.3: subr.el patch --]
[-- Type: text/x-patch, Size: 753 bytes --]
*** /home/steve/cvsroot/emacs/lisp/subr.el.~1.623.~ 2008-11-20 13:49:52.000000000 +0100
--- /home/steve/cvsroot/emacs/lisp/subr.el 2008-12-12 15:04:49.000000000 +0100
***************
*** 2248,2254 ****
(play-sound-internal sound)
(error "This Emacs binary lacks sound support")))
! (declare-function w32-shell-dos-semantics "w32-fns" nil)
(defun shell-quote-argument (argument)
"Quote an argument for passing as argument to an inferior shell."
--- 2248,2254 ----
(play-sound-internal sound)
(error "This Emacs binary lacks sound support")))
! (declare-function w32-shell-dos-semantics "w32-fns")
(defun shell-quote-argument (argument)
"Quote an argument for passing as argument to an inferior shell."
[-- Warning: decoded text below may be mangled, UTF-8 assumed --]
[-- Attachment #2.1.4: todo-mode.el patch --]
[-- Type: text/x-patch, Size: 694 bytes --]
*** /home/steve/cvsroot/emacs/lisp/calendar/todo-mode.el.~1.71.~ 2008-10-25 20:48:16.000000000 +0200
--- /home/steve/cvsroot/emacs/lisp/calendar/todo-mode.el 2008-12-12 14:57:25.000000000 +0100
***************
*** 911,917 ****
(defvar entry)
;; t-c should be used from diary code, which requires calendar.
! (declare-function calendar-current-date "calendar" nil)
;; Read about this function in the setup instructions above!
;;;###autoload
--- 911,917 ----
(defvar entry)
;; t-c should be used from diary code, which requires calendar.
! (declare-function calendar-current-date "calendar")
;; Read about this function in the setup instructions above!
;;;###autoload
[-- Warning: decoded text below may be mangled, UTF-8 assumed --]
[-- Attachment #2.1.5: fortran.el patch --]
[-- Type: text/x-patch, Size: 745 bytes --]
*** /home/steve/cvsroot/emacs/lisp/progmodes/fortran.el.~1.151.~ 2008-11-20 13:49:53.000000000 +0100
--- /home/steve/cvsroot/emacs/lisp/progmodes/fortran.el 2008-12-12 14:59:26.000000000 +0100
***************
*** 914,920 ****
"Fortran mode adds this to `hack-local-variables-hook'."
(fortran-line-length fortran-line-length))
! (declare-function gud-find-c-expr "gud.el" nil)
(defun fortran-gud-find-expr ()
;; Consider \n as punctuation (end of expression).
--- 914,920 ----
"Fortran mode adds this to `hack-local-variables-hook'."
(fortran-line-length fortran-line-length))
! (declare-function gud-find-c-expr "gud.el")
(defun fortran-gud-find-expr ()
;; Consider \n as punctuation (end of expression).
[-- Attachment #3: Type: message/rfc822, Size: 2124 bytes --]
From: Glenn Morris <rgm@gnu.org>
To: 1551-done@emacsbugs.donarmstrong.com
Subject: Re: bug#1551: 23.0.60; declare-function: redundant `nil' arg
Date: Fri, 12 Dec 2008 14:08:45 -0500
Message-ID: <078wqlky82.fsf@fencepost.gnu.org>
tags 1551 notabug
stop
Stephen Berman wrote:
> According to the argument-list of declare-function, only its first two
> arguments are obligatory. But by rgrepping emacs/lisp for
> "(declare-function" I found four invocations with `nil' as the third
> argument (there are many more invocations with only two arguments):
Not a bug. Third argument unspecified means argument list unspecified.
Third argument nil means argument list stated to be nil. These are not
the same thing.
^ permalink raw reply [flat|nested] 3+ messages in thread
end of thread, other threads:[~2008-12-12 19:15 UTC | newest]
Thread overview: 3+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
[not found] <078wqlky82.fsf@fencepost.gnu.org>
2008-12-12 14:23 ` bug#1551: 23.0.60; declare-function: redundant `nil' arg Stephen Berman
2008-12-12 19:15 ` bug#1551: marked as done (23.0.60; declare-function: redundant `nil' arg) Emacs bug Tracking System
2008-12-12 19:15 ` Processed: Re: bug#1551: 23.0.60; declare-function: redundant `nil' arg Emacs bug Tracking System
Code repositories for project(s) associated with this external index
https://git.savannah.gnu.org/cgit/emacs.git
https://git.savannah.gnu.org/cgit/emacs/org-mode.git
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.