* SIGTYPE/RETSIGTYPE
@ 2011-03-26 19:34 Glenn Morris
2011-03-27 1:14 ` SIGTYPE/RETSIGTYPE Paul Eggert
0 siblings, 1 reply; 4+ messages in thread
From: Glenn Morris @ 2011-03-26 19:34 UTC (permalink / raw)
To: emacs-devel
The autoconf manual says:
Obsolete Macros: AC_TYPE_SIGNAL
If signal.h declares signal as returning a pointer to a function
returning void, define RETSIGTYPE to be void; otherwise, define it
to be int. These days, it is portable to assume C89, and that signal
handlers return void, without needing to use this macro or RETSIGTYPE.
So is it OK to remove this test from Emacs's configure.in, and replace
RETSIGTYPE (and SIGTYPE, which we define equal to RETSIGTYPE in
AH_BOTTOM) with just `void' everywhere in the sources?
^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: SIGTYPE/RETSIGTYPE
2011-03-26 19:34 SIGTYPE/RETSIGTYPE Glenn Morris
@ 2011-03-27 1:14 ` Paul Eggert
2011-03-27 2:30 ` SIGTYPE/RETSIGTYPE Glenn Morris
0 siblings, 1 reply; 4+ messages in thread
From: Paul Eggert @ 2011-03-27 1:14 UTC (permalink / raw)
To: Glenn Morris; +Cc: emacs-devel
On 03/26/2011 12:34 PM, Glenn Morris wrote:
> So is it OK to remove this test from Emacs's configure.in, and replace
> RETSIGTYPE (and SIGTYPE, which we define equal to RETSIGTYPE in
> AH_BOTTOM) with just `void' everywhere in the sources?
Yes, absolutely. Other GNU projects have done that
for years, with no problems.
^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: SIGTYPE/RETSIGTYPE
2011-03-27 1:14 ` SIGTYPE/RETSIGTYPE Paul Eggert
@ 2011-03-27 2:30 ` Glenn Morris
2011-03-27 3:09 ` SIGTYPE/RETSIGTYPE Paul Eggert
0 siblings, 1 reply; 4+ messages in thread
From: Glenn Morris @ 2011-03-27 2:30 UTC (permalink / raw)
To: Paul Eggert; +Cc: emacs-devel
Thanks; removed.
Now the only things that `autoreconf -Wall' still gripes about are in
lib/ and m4/. ;)
^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: SIGTYPE/RETSIGTYPE
2011-03-27 2:30 ` SIGTYPE/RETSIGTYPE Glenn Morris
@ 2011-03-27 3:09 ` Paul Eggert
0 siblings, 0 replies; 4+ messages in thread
From: Paul Eggert @ 2011-03-27 3:09 UTC (permalink / raw)
To: Glenn Morris; +Cc: emacs-devel
On 03/26/2011 07:30 PM, Glenn Morris wrote:
> Now the only things that `autoreconf -Wall' still gripes about are in
> lib/ and m4/. ;)
The gripes about AC_GNU_SOURCE are because gnulib is still
intended to be portable to Autoconf 2.59, which doesn't have
the replacements for AC_GNU_SOURCE. These gripes should go
away once gnulib is modernized to assume a later Autoconf.
The other gripe, about DEFAULT_INCLUDES, is due to the way
Emacs historically has put config.h into src. Most other
packages put it into lib, and if we did that we could
remove the DEFAULT_INCLUDES bit and fix the gripe.
^ permalink raw reply [flat|nested] 4+ messages in thread
end of thread, other threads:[~2011-03-27 3:09 UTC | newest]
Thread overview: 4+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2011-03-26 19:34 SIGTYPE/RETSIGTYPE Glenn Morris
2011-03-27 1:14 ` SIGTYPE/RETSIGTYPE Paul Eggert
2011-03-27 2:30 ` SIGTYPE/RETSIGTYPE Glenn Morris
2011-03-27 3:09 ` SIGTYPE/RETSIGTYPE Paul Eggert
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.