[src/trunk]: src/lib/libcurses curses: use perror rather than err in initscr

[roy <roy%NetBSD.org@localhost>]

details:   https://anonhg.NetBSD.org/src/rev/d92cc978a3c7
branches:  trunk
changeset: 745790:d92cc978a3c7
user:      roy <roy%NetBSD.org@localhost>
date:      Thu Mar 12 15:50:11 2020 +0000

description:
curses: use perror rather than err in initscr

libhack lacks err and perror is more portable.

diffstat:

 lib/libcurses/initscr.c |  16 +++++++++++-----
 1 files changed, 11 insertions(+), 5 deletions(-)

diffs (40 lines):

diff -r 24ba87f519de -r d92cc978a3c7 lib/libcurses/initscr.c
--- a/lib/libcurses/initscr.c   Thu Mar 12 15:04:13 2020 +0000
+++ b/lib/libcurses/initscr.c   Thu Mar 12 15:50:11 2020 +0000
@@ -1,4 +1,4 @@
-/*     $NetBSD: initscr.c,v 1.34 2020/03/11 21:33:38 roy Exp $ */
+/*     $NetBSD: initscr.c,v 1.35 2020/03/12 15:50:11 roy Exp $ */
 
 /*
  * Copyright (c) 1981, 1993, 1994
@@ -34,11 +34,10 @@
 #if 0
 static char sccsid[] = "@(#)initscr.c  8.2 (Berkeley) 5/4/94";
 #else
-__RCSID("$NetBSD: initscr.c,v 1.34 2020/03/11 21:33:38 roy Exp $");
+__RCSID("$NetBSD: initscr.c,v 1.35 2020/03/12 15:50:11 roy Exp $");
 #endif
 #endif /* not lint */
 
-#include <err.h>
 #include <stdlib.h>
 
 #include "curses.h"
@@ -66,8 +65,15 @@
                sp = Def_term;
 
        /* LINTED const castaway; newterm does not modify sp! */
-       if ((_cursesi_screen = newterm((char *) sp, stdout, stdin)) == NULL)
-               errx(EXIT_FAILURE, "initscr"); /* POSIX says exit on failure */
+       if ((_cursesi_screen = newterm((char *) sp, stdout, stdin)) == NULL) {
+               /*
+                * POSIX says we should write a diagnostic and exit on error.
+                * As such some applications don't bother checking the return
+                * value at all.
+                */
+               perror("initscr");
+               exit(EXIT_FAILURE);
+       }
 
        set_term(_cursesi_screen);
        wrefresh(curscr);