circlecloud/tmux
1
0
Fork 0
mirror of https://github.com/tmux/tmux.git synced 2025-11-04 09:26:05 +00:00
Code Issues Projects Releases Wiki Activity

Do fatal/fatalx a different way so the compiler trick to avoid warnings

Browse Source 
becomes unnecessary, prompted by theo.
This commit is contained in:
nicm
2021-09-10 15:03:18 +00:00
parent 33ac7a346e
commit e6b40cb339
2 changed files with 17 additions and 32 deletions
Download Patch File Download Diff File Expand all files Collapse all files

35
log.c
View File

@@ -99,28 +99,27 @@ log_close(void)
/* Write a log message. */ /* Write a log message. */
static void printflike(1, 0) static void printflike(1, 0)
log_vwrite(const char *msg, va_list ap) log_vwrite(const char *msg, va_list ap, const char *prefix)
{ {
char *fmt, *out; char *s, *out;
struct timeval tv; struct timeval tv;
if (log_file == NULL) if (log_file == NULL)
return; return;
if (vasprintf(&fmt, msg, ap) == -1) if (vasprintf(&s, msg, ap) == -1)
return; return;
if (stravis(&out, fmt, VIS_OCTAL|VIS_CSTYLE|VIS_TAB|VIS_NL) == -1) { if (stravis(&out, s, VIS_OCTAL|VIS_CSTYLE|VIS_TAB|VIS_NL) == -1) {
free(fmt); free(s);
return; return;
} }
free(s);
gettimeofday(&tv, NULL); gettimeofday(&tv, NULL);
if (fprintf(log_file, "%lld.%06d %s\n", (long long)tv.tv_sec, if (fprintf(log_file, "%lld.%06d %s%s\n", (long long)tv.tv_sec,
(int)tv.tv_usec, out) != -1) (int)tv.tv_usec, prefix, out) != -1)
fflush(log_file); fflush(log_file);
free(out); free(out);
free(fmt);
} }
/* Log a debug message. */ /* Log a debug message. */
@@ -133,7 +132,7 @@ log_debug(const char *msg, ...)
return; return;
va_start(ap, msg); va_start(ap, msg);
log_vwrite(msg, ap); log_vwrite(msg, ap, "");
va_end(ap); va_end(ap);
} }
@@ -141,14 +140,16 @@ log_debug(const char *msg, ...)
__dead void __dead void
fatal(const char *msg, ...) fatal(const char *msg, ...)
{ {
char *fmt; char tmp[256];
va_list ap; va_list ap;
if (snprintf(tmp, sizeof tmp, "fatal: %s: ", strerror(errno)) < 0)
exit (1);
va_start(ap, msg); va_start(ap, msg);
if (asprintf(&fmt, "fatal: %s: %s", msg, strerror(errno)) == -1) log_vwrite(msg, ap, tmp);
exit(1);
no_format_nonliteral(log_vwrite(fmt, ap));
va_end(ap); va_end(ap);
exit(1); exit(1);
} }
@@ -156,13 +157,11 @@ fatal(const char *msg, ...)
__dead void __dead void
fatalx(const char *msg, ...) fatalx(const char *msg, ...)
{ {
char *fmt;
va_list ap; va_list ap;
va_start(ap, msg); va_start(ap, msg);
if (asprintf(&fmt, "fatal: %s", msg) == -1) log_vwrite(msg, ap, "fatal: ");
exit(1);
no_format_nonliteral(log_vwrite(fmt, ap));
va_end(ap); va_end(ap);
exit(1); exit(1);
} }

14
tmux.h
View File

@@ -93,20 +93,6 @@ struct winlink;
#define DEFAULT_XPIXEL 16 #define DEFAULT_XPIXEL 16
#define DEFAULT_YPIXEL 32 #define DEFAULT_YPIXEL 32
/* Don't complain about format arguments. */
#if __clang__ || __GNUC__ > 4 || (__GNUC__ == 4 && __GNUC_MINOR__ >= 6)
#define no_format_nonliteral(x) do { \
_Pragma ("GCC diagnostic push") \
_Pragma ("GCC diagnostic ignored \"-Wformat-nonliteral\"") \
x; \
_Pragma ("GCC diagnostic pop") \
} while (0)
#else
#define no_format_nonliteral(x) do { \
x; \
} while (0)
#endif
/* Attribute to make GCC check printf-like arguments. */ /* Attribute to make GCC check printf-like arguments. */
#define printflike(a, b) __attribute__ ((format (printf, a, b))) #define printflike(a, b) __attribute__ ((format (printf, a, b)))