Okay to push this patch?

As far as I can tell from GitHub code search setchrclass was only an
IRIX thing. Looks like dead code to me anyways since we don't have a
configure check for the function.

Likewise, VRPRNT seems to be an IRIX thing. And now that Gnulib does not
support it, missing this will be the least of their issues.

Collin

>From 321f73240c74f5414820f5e0e410309257df1cb9 Mon Sep 17 00:00:00 2001
Message-ID: <321f73240c74f5414820f5e0e410309257df1cb9.1759458133.git.collin.fu...@gmail.com>
From: Collin Funk <[email protected]>
Date: Thu, 2 Oct 2025 19:15:34 -0700
Subject: [PATCH] maint: remove IRIX support

* src/ptx.c (main) [HAVE_SETCHRCLASS]: Remove call to setchrclass.
* src/stty.c (VREPRINT) [!VREPRINT && VRPRNT]: Remove definition.
---
 src/ptx.c  | 4 ----
 src/stty.c | 3 ---
 2 files changed, 7 deletions(-)

diff --git a/src/ptx.c b/src/ptx.c
index 1f7fc6731..463f508c7 100644
--- a/src/ptx.c
+++ b/src/ptx.c
@@ -1780,10 +1780,6 @@ main (int argc, char **argv)
 
   atexit (close_stdout);
 
-#if HAVE_SETCHRCLASS
-  setchrclass (nullptr);
-#endif
-
   while (optchar = getopt_long (argc, argv, "AF:GM:ORS:TW:b:i:fg:o:trw:",
                                 long_options, nullptr),
          optchar != EOF)
diff --git a/src/stty.c b/src/stty.c
index c9523efdc..919e32bb3 100644
--- a/src/stty.c
+++ b/src/stty.c
@@ -131,9 +131,6 @@
 #if defined VDSUSP && !defined CDSUSP
 # define CDSUSP Control ('y')
 #endif
-#if !defined VREPRINT && defined VRPRNT /* Irix 4.0.5 */
-# define VREPRINT VRPRNT
-#endif
 #if defined VREPRINT && !defined CRPRNT
 # define CRPRNT Control ('r')
 #endif
-- 
2.51.0

Reply via email to