On 5/11/22 03:27, Reuben Thomas wrote:
Bruno handled alloca.c, and I did parse-datetime.y with the attached. Thanks for reporting it.Mostly in alloca.c, with one case in parse-datetime.y.
From 950f04bbf18dad544c61f448206e9dc96cbe3b7a Mon Sep 17 00:00:00 2001 From: Paul Eggert <egg...@cs.ucla.edu> Date: Wed, 11 May 2022 09:35:45 -0700 Subject: [PATCH] parse-datetime: remove Emacs cruft MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit
* lib/parse-datetime.y: Remove an ‘ifdef emacs’. Emacs has never used this module. The module is derived from code taken from Emacs, but that code was removed from Emacs in the 1990s. --- ChangeLog | 7 +++++++ lib/parse-datetime.y | 10 ---------- 2 files changed, 7 insertions(+), 10 deletions(-) diff --git a/ChangeLog b/ChangeLog index f4e24d4157..8cc17d8d14 100644 --- a/ChangeLog +++ b/ChangeLog @@ -1,3 +1,10 @@ +2022-05-11 Paul Eggert <egg...@cs.ucla.edu> + + parse-datetime: remove Emacs cruft + * lib/parse-datetime.y: Remove an ‘ifdef emacs’. Emacs has never + used this module. The module is derived from code taken from + Emacs, but that code was removed from Emacs in the 1990s. + 2022-05-11 Bruno Haible <br...@clisp.org> alloca: Remove old code for Emacs, unused since 2009. diff --git a/lib/parse-datetime.y b/lib/parse-datetime.y index 9fc14c9d46..7220d05dd7 100644 --- a/lib/parse-datetime.y +++ b/lib/parse-datetime.y @@ -52,16 +52,6 @@ #define YYMAXDEPTH 20 #define YYINITDEPTH YYMAXDEPTH -/* Since the code of parse-datetime.y is not included in the Emacs executable - itself, there is no need to #define static in this file. Even if - the code were included in the Emacs executable, it probably - wouldn't do any harm to #undef it here; this will only cause - problems if we try to write to a static variable, which I don't - think this code needs to do. */ -#ifdef emacs -# undef static -#endif - #include <inttypes.h> #include <c-ctype.h> #include <stdarg.h> -- 2.34.1