Module Name: xsrc Committed By: christos Date: Mon May 10 12:29:12 UTC 2021
Modified Files: xsrc/external/mit/xterm/dist: misc.c xsrc/external/mit/xterm/include: xtermcfg.h Log Message: revert previous, do it differently and explain why. To generate a diff of this commit: cvs rdiff -u -r1.19 -r1.20 xsrc/external/mit/xterm/dist/misc.c cvs rdiff -u -r1.17 -r1.18 xsrc/external/mit/xterm/include/xtermcfg.h Please note that diffs are not public domain; they are subject to the copyright notices on the relevant files.
Modified files: Index: xsrc/external/mit/xterm/dist/misc.c diff -u xsrc/external/mit/xterm/dist/misc.c:1.19 xsrc/external/mit/xterm/dist/misc.c:1.20 --- xsrc/external/mit/xterm/dist/misc.c:1.19 Sun May 9 12:55:36 2021 +++ xsrc/external/mit/xterm/dist/misc.c Mon May 10 08:29:11 2021 @@ -6563,7 +6563,7 @@ xioerror(Display *dpy) return 0; /* appease the compiler */ } -void _X_NORETURN +void xt_error(String message) { xtermWarning("Xt error: %s\n", message); Index: xsrc/external/mit/xterm/include/xtermcfg.h diff -u xsrc/external/mit/xterm/include/xtermcfg.h:1.17 xsrc/external/mit/xterm/include/xtermcfg.h:1.18 --- xsrc/external/mit/xterm/include/xtermcfg.h:1.17 Mon Apr 26 22:11:31 2021 +++ xsrc/external/mit/xterm/include/xtermcfg.h Mon May 10 08:29:11 2021 @@ -98,7 +98,16 @@ #define HAVE_SETPGID 1 /* AC_CHECK_FUNCS(setpgid) */ #define HAVE_STDINT_H 1 /* AC_PROG_CC_STDC */ #define HAVE_STDLIB_H 1 /* AC_CHECK_HEADERS(stdlib.h) */ +#if 0 +/* + * clang treats _Noreturn and __attribute__((__noreturn__)) differently, + * so we end up labeling xt_error() with _Noreturn and using it in + * XtSetErrorHandler which is marked with the attribute noreturn, and clang + * complains. IMHO this is a bug in clang. We tell everyone that we don't + * have the header, so we use the attribute consistently and everyone is happy. + */ #define HAVE_STDNORETURN_H 1 /* CF_C11_NORETURN */ +#endif #define HAVE_STRFTIME 1 /* AC_CHECK_FUNCS(strftime) */ /* #undef HAVE_SYS_TIME_H */ /* AC_HEADER_TIME */ #define HAVE_SYS_TTYDEFAULTS_H 1 /* AC_CHECK_HEADERS(sys/ttydefaults.h) */