Tidy logging and lose stdout hacks for _info.

pull/1/head
Nicholas Marriott 2008-08-08 17:35:42 +00:00
parent de0e1c6284
commit 8ed403e309
3 changed files with 90 additions and 67 deletions

138
log.c
View File

@ -1,4 +1,4 @@
/* $Id: log.c,v 1.4 2007-10-19 22:17:29 nicm Exp $ */ /* $Id: log.c,v 1.5 2008-08-08 17:35:42 nicm Exp $ */
/* /*
* Copyright (c) 2007 Nicholas Marriott <nicm@users.sourceforge.net> * Copyright (c) 2007 Nicholas Marriott <nicm@users.sourceforge.net>
@ -23,79 +23,110 @@
#include <stdlib.h> #include <stdlib.h>
#include <string.h> #include <string.h>
#include <syslog.h> #include <syslog.h>
#include <time.h>
#include "tmux.h" #include "tmux.h"
/* Logging enabled. */ /* Logging type. */
int log_enabled; #define LOG_TYPE_OFF 0
#define LOG_TYPE_SYSLOG 1
#define LOG_TYPE_TTY 2
#define LOG_TYPE_FILE 3
int log_type = LOG_TYPE_OFF;
/* Log stream or NULL to use syslog. */ /* Log file, if needed. */
FILE *log_stream; FILE *log_file;
/* Debug level. */ /* Debug level. */
int log_level; int log_level;
/* Open logging. */ /* Open logging to syslog. */
void void
log_open(FILE *f, int facility, int level) log_open_syslog(int level)
{ {
log_stream = f; log_type = LOG_TYPE_SYSLOG;
if (f != NULL)
setlinebuf(f);
log_level = level; log_level = level;
if (f == NULL) openlog(__progname, LOG_PID|LOG_NDELAY, LOG_FACILITY);
openlog(__progname, LOG_PID|LOG_NDELAY, facility);
tzset();
log_enabled = 1; tzset();
}
/* Open logging to tty. */
void
log_open_tty(int level)
{
log_type = LOG_TYPE_TTY;
log_level = level;
setlinebuf(stderr);
setlinebuf(stdout);
tzset();
}
/* Open logging to tty. */
void
log_open_file(int level, const char *path)
{
log_file = fopen(path, "w");
if (log_file == NULL)
return;
log_type = LOG_TYPE_FILE;
log_level = level;
setlinebuf(log_file);
tzset();
} }
/* Close logging. */ /* Close logging. */
void void
log_close(void) log_close(void)
{ {
if (log_stream != NULL && log_stream != stderr) /* XXX */ if (log_type == LOG_TYPE_FILE)
fclose(log_stream); fclose(log_file);
log_enabled = 0; log_type = LOG_TYPE_OFF;
} }
/* Write a log message. */ /* Write a log message. */
void void
log_write(FILE *f, int priority, const char *msg, ...) log_write(int pri, const char *msg, ...)
{ {
va_list ap; va_list ap;
va_start(ap, msg); va_start(ap, msg);
log_vwrite(f, priority, msg, ap); log_vwrite(pri, msg, ap);
va_end(ap); va_end(ap);
} }
/* Write a log message. */ /* Write a log message. */
void void
log_vwrite(FILE *f, int priority, const char *msg, va_list ap) log_vwrite(int pri, const char *msg, va_list ap)
{ {
char *fmt; char *fmt;
FILE *f = log_file;
if (!log_enabled) switch (log_type) {
return; case LOG_TYPE_SYSLOG:
vsyslog(pri, msg, ap);
if (f == NULL) break;
f = log_stream; case LOG_TYPE_TTY:
if (f == NULL) { if (pri == LOG_INFO)
vsyslog(priority, msg, ap); f = stdout;
return; else
f = stderr;
/* FALLTHROUGH */
case LOG_TYPE_FILE:
if (asprintf(&fmt, "%s\n", msg) == -1)
exit(1);
if (vfprintf(f, fmt, ap) == -1)
exit(1);
fflush(f);
free(fmt);
break;
} }
if (asprintf(&fmt, "%s\n", msg) == -1)
exit(1);
if (vfprintf(f, fmt, ap) == -1)
exit(1);
fflush(f);
free(fmt);
} }
/* Log a warning with error string. */ /* Log a warning with error string. */
@ -105,13 +136,10 @@ log_warn(const char *msg, ...)
va_list ap; va_list ap;
char *fmt; char *fmt;
if (!log_enabled)
return;
va_start(ap, msg); va_start(ap, msg);
if (asprintf(&fmt, "%s: %s", msg, strerror(errno)) == -1) if (asprintf(&fmt, "%s: %s", msg, strerror(errno)) == -1)
exit(1); exit(1);
log_vwrite(NULL, LOG_CRIT, fmt, ap); log_vwrite(LOG_CRIT, fmt, ap);
free(fmt); free(fmt);
va_end(ap); va_end(ap);
} }
@ -123,7 +151,7 @@ log_warnx(const char *msg, ...)
va_list ap; va_list ap;
va_start(ap, msg); va_start(ap, msg);
log_vwrite(NULL, LOG_CRIT, msg, ap); log_vwrite(LOG_CRIT, msg, ap);
va_end(ap); va_end(ap);
} }
@ -135,10 +163,7 @@ log_info(const char *msg, ...)
if (log_level > -1) { if (log_level > -1) {
va_start(ap, msg); va_start(ap, msg);
if (log_stream == stderr) /* XXX */ log_vwrite(LOG_INFO, msg, ap);
log_vwrite(stdout, LOG_INFO, msg, ap);
else
log_vwrite(NULL, LOG_INFO, msg, ap);
va_end(ap); va_end(ap);
} }
} }
@ -151,7 +176,7 @@ log_debug(const char *msg, ...)
if (log_level > 0) { if (log_level > 0) {
va_start(ap, msg); va_start(ap, msg);
log_vwrite(NULL, LOG_DEBUG, msg, ap); log_vwrite(LOG_DEBUG, msg, ap);
va_end(ap); va_end(ap);
} }
} }
@ -164,7 +189,7 @@ log_debug2(const char *msg, ...)
if (log_level > 1) { if (log_level > 1) {
va_start(ap, msg); va_start(ap, msg);
log_vwrite(NULL, LOG_DEBUG, msg, ap); log_vwrite(LOG_DEBUG, msg, ap);
va_end(ap); va_end(ap);
} }
} }
@ -177,7 +202,7 @@ log_debug3(const char *msg, ...)
if (log_level > 2) { if (log_level > 2) {
va_start(ap, msg); va_start(ap, msg);
log_vwrite(NULL, LOG_DEBUG, msg, ap); log_vwrite(LOG_DEBUG, msg, ap);
va_end(ap); va_end(ap);
} }
} }
@ -188,17 +213,14 @@ log_vfatal(const char *msg, va_list ap)
{ {
char *fmt; char *fmt;
if (!log_enabled)
exit(1);
if (errno != 0) { if (errno != 0) {
if (asprintf(&fmt, "fatal: %s: %s", msg, strerror(errno)) == -1) if (asprintf(&fmt, "fatal: %s: %s", msg, strerror(errno)) == -1)
exit(1); exit(1);
log_vwrite(NULL, LOG_CRIT, fmt, ap); log_vwrite(LOG_CRIT, fmt, ap);
} else { } else {
if (asprintf(&fmt, "fatal: %s", msg) == -1) if (asprintf(&fmt, "fatal: %s", msg) == -1)
exit(1); exit(1);
log_vwrite(NULL, LOG_CRIT, fmt, ap); log_vwrite(LOG_CRIT, fmt, ap);
} }
free(fmt); free(fmt);
@ -206,7 +228,7 @@ log_vfatal(const char *msg, va_list ap)
} }
/* Log a critical error, with error string, and die. */ /* Log a critical error, with error string, and die. */
__dead void __dead void printflike1
log_fatal(const char *msg, ...) log_fatal(const char *msg, ...)
{ {
va_list ap; va_list ap;
@ -216,7 +238,7 @@ log_fatal(const char *msg, ...)
} }
/* Log a critical error and die. */ /* Log a critical error and die. */
__dead void __dead void printflike1
log_fatalx(const char *msg, ...) log_fatalx(const char *msg, ...)
{ {
va_list ap; va_list ap;

8
tmux.c
View File

@ -1,4 +1,4 @@
/* $Id: tmux.c,v 1.72 2008-07-02 21:22:57 nicm Exp $ */ /* $Id: tmux.c,v 1.73 2008-08-08 17:35:42 nicm Exp $ */
/* /*
* Copyright (c) 2007 Nicholas Marriott <nicm@users.sourceforge.net> * Copyright (c) 2007 Nicholas Marriott <nicm@users.sourceforge.net>
@ -71,15 +71,13 @@ usage(void)
void void
logfile(const char *name) logfile(const char *name)
{ {
FILE *f;
char *path; char *path;
log_close(); log_close();
if (debug_level > 0) { if (debug_level > 0) {
xasprintf( xasprintf(
&path, "%s-%s-%ld.log", __progname, name, (long) getpid()); &path, "%s-%s-%ld.log", __progname, name, (long) getpid());
f = fopen(path, "w"); log_open_file(debug_level, path);
log_open(f, LOG_DAEMON, debug_level);
xfree(path); xfree(path);
} }
} }
@ -205,7 +203,7 @@ main(int argc, char **argv)
argc -= optind; argc -= optind;
argv += optind; argv += optind;
log_open(stderr, LOG_USER, debug_level); log_open_tty(debug_level);
siginit(); siginit();
options_init(&global_options, NULL); options_init(&global_options, NULL);

11
tmux.h
View File

@ -1,4 +1,4 @@
/* $Id: tmux.h,v 1.180 2008-08-07 20:20:52 nicm Exp $ */ /* $Id: tmux.h,v 1.181 2008-08-08 17:35:42 nicm Exp $ */
/* /*
* Copyright (c) 2007 Nicholas Marriott <nicm@users.sourceforge.net> * Copyright (c) 2007 Nicholas Marriott <nicm@users.sourceforge.net>
@ -1295,10 +1295,13 @@ int buffer_poll(struct pollfd *, struct buffer *, struct buffer *);
void buffer_flush(int, struct buffer *n, struct buffer *); void buffer_flush(int, struct buffer *n, struct buffer *);
/* log.c */ /* log.c */
void log_open(FILE *, int, int); #define LOG_FACILITY LOG_DAEMON
void log_open_syslog(int);
void log_open_tty(int);
void log_open_file(int, const char *);
void log_close(void); void log_close(void);
void log_vwrite(FILE *, int, const char *, va_list); void log_vwrite(int, const char *, va_list);
void log_write(FILE *, int, const char *, ...); void log_write(int, const char *, ...);
void printflike1 log_warn(const char *, ...); void printflike1 log_warn(const char *, ...);
void printflike1 log_warnx(const char *, ...); void printflike1 log_warnx(const char *, ...);
void printflike1 log_info(const char *, ...); void printflike1 log_info(const char *, ...);