On 5/11/22 03:27, Reuben Thomas wrote:
Mostly in alloca.c, with one case in parse-datetime.y.
Bruno handled alloca.c, and I did parse-datetime.y with the attached. Thanks for reporting it.
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

Reply via email to