The branch, master has been updated
       via  b1a87b2ee49028f7a35bc39db47833a82cd74e38 (commit)
      from  c5a30513edc6f108b6387ee9cf6f1cb9dc7a674c (commit)

- Log -----------------------------------------------------------------
commit b1a87b2ee49028f7a35bc39db47833a82cd74e38
Author: Nicholas Marriott <nicholas.marri...@gmail.com>
Commit: Nicholas Marriott <nicholas.marri...@gmail.com>

    Remove log_debug2 as well and simplify log.c.
---
 input-keys.c |    6 ++--
 log.c        |   77 ++++++++++++++++------------------------------------------
 tmux.c       |    2 +-
 tmux.h       |    3 +-
 4 files changed, 26 insertions(+), 62 deletions(-)

diff --git a/input-keys.c b/input-keys.c
index 7582a63..46c37d0 100644
--- a/input-keys.c
+++ b/input-keys.c
@@ -142,7 +142,7 @@ input_key(struct window_pane *wp, int key)
        char                           *out;
        u_char                          ch;
 
-       log_debug2("writing key 0x%x", key);
+       log_debug("writing key 0x%x", key);
 
        /*
         * If this is a normal 7-bit key, just send it, with a leading escape
@@ -185,11 +185,11 @@ input_key(struct window_pane *wp, int key)
                        break;
        }
        if (i == nitems(input_keys)) {
-               log_debug2("key 0x%x missing", key);
+               log_debug("key 0x%x missing", key);
                return;
        }
        dlen = strlen(ike->data);
-       log_debug2("found key 0x%x: \"%s\"", key, ike->data);
+       log_debug("found key 0x%x: \"%s\"", key, ike->data);
 
        /* Prefix a \033 for escape. */
        if (key & KEYC_ESCAPE)
diff --git a/log.c b/log.c
index 71cedde..7ef70ab 100644
--- a/log.c
+++ b/log.c
@@ -22,20 +22,13 @@
 #include <stdio.h>
 #include <stdlib.h>
 #include <string.h>
-#include <syslog.h>
-#include <time.h>
 
 #include "tmux.h"
 
-/* Log file, if needed. */
-FILE           *log_file;
+FILE   *log_file;
 
-/* Debug level. */
-int             log_level = 0;
-
-void            log_event_cb(int, const char *);
-void            log_vwrite(const char *, va_list);
-__dead void     log_vfatal(const char *, va_list);
+void    log_event_cb(int, const char *);
+void    log_vwrite(const char *, va_list);
 
 /* Log callback for libevent. */
 void
@@ -46,12 +39,11 @@ log_event_cb(unused int severity, const char *msg)
 
 /* Open logging to file. */
 void
-log_open(int level, const char *path)
+log_open(const char *path)
 {
        log_file = fopen(path, "w");
        if (log_file == NULL)
                return;
-       log_level = level;
 
        setlinebuf(log_file);
        event_set_log_callback(log_event_cb);
@@ -65,6 +57,7 @@ log_close(void)
 {
        if (log_file != NULL)
                fclose(log_file);
+       log_file = NULL;
 
        event_set_log_callback(NULL);
 }
@@ -92,63 +85,35 @@ log_debug(const char *msg, ...)
 {
        va_list ap;
 
-       if (log_level > 0) {
-               va_start(ap, msg);
-               log_vwrite(msg, ap);
-               va_end(ap);
-       }
-}
-
-/* Log a debug message at level 2. */
-void printflike1
-log_debug2(const char *msg, ...)
-{
-       va_list ap;
-
-       if (log_level > 1) {
-               va_start(ap, msg);
-               log_vwrite(msg, ap);
-               va_end(ap);
-       }
-}
-
-/* Log a critical error, with error string if necessary, and die. */
-__dead void
-log_vfatal(const char *msg, va_list ap)
-{
-       char    *fmt;
-
-       if (errno != 0) {
-               if (asprintf(&fmt, "fatal: %s: %s", msg, strerror(errno)) == -1)
-                       exit(1);
-               log_vwrite(fmt, ap);
-       } else {
-               if (asprintf(&fmt, "fatal: %s", msg) == -1)
-                       exit(1);
-               log_vwrite(fmt, ap);
-       }
-       free(fmt);
-
-       exit(1);
+       va_start(ap, msg);
+       log_vwrite(msg, ap);
+       va_end(ap);
 }
 
-/* Log a critical error, with error string, and die. */
+/* Log a critical error with error string and die. */
 __dead void printflike1
 log_fatal(const char *msg, ...)
 {
-       va_list ap;
+       char    *fmt;
+       va_list  ap;
 
        va_start(ap, msg);
-       log_vfatal(msg, ap);
+       if (asprintf(&fmt, "fatal: %s: %s", msg, strerror(errno)) == -1)
+               exit(1);
+       log_vwrite(fmt, ap);
+       exit(1);
 }
 
 /* Log a critical error and die. */
 __dead void printflike1
 log_fatalx(const char *msg, ...)
 {
-       va_list ap;
+       char    *fmt;
+       va_list  ap;
 
-       errno = 0;
        va_start(ap, msg);
-       log_vfatal(msg, ap);
+       if (asprintf(&fmt, "fatal: %s", msg) == -1)
+               exit(1);
+       log_vwrite(fmt, ap);
+       exit(1);
 }
diff --git a/tmux.c b/tmux.c
index a804e6f..7374dd5 100644
--- a/tmux.c
+++ b/tmux.c
@@ -73,7 +73,7 @@ logfile(const char *name)
 
        if (debug_level > 0) {
                xasprintf(&path, "tmux-%s-%ld.log", name, (long) getpid());
-               log_open(debug_level, path);
+               log_open(path);
                free(path);
        }
 }
diff --git a/tmux.h b/tmux.h
index a9a1e08..5aac390 100644
--- a/tmux.h
+++ b/tmux.h
@@ -2330,10 +2330,9 @@ char             *osdep_get_cwd(int);
 struct event_base *osdep_event_init(void);
 
 /* log.c */
-void            log_open(int, const char *);
+void            log_open(const char *);
 void            log_close(void);
 void printflike1 log_debug(const char *, ...);
-void printflike1 log_debug2(const char *, ...);
 __dead void printflike1 log_fatal(const char *, ...);
 __dead void printflike1 log_fatalx(const char *, ...);
 


-----------------------------------------------------------------------

Summary of changes:
 input-keys.c |    6 ++--
 log.c        |   77 ++++++++++++++++------------------------------------------
 tmux.c       |    2 +-
 tmux.h       |    3 +-
 4 files changed, 26 insertions(+), 62 deletions(-)


hooks/post-receive
-- 
tmux

------------------------------------------------------------------------------
Subversion Kills Productivity. Get off Subversion & Make the Move to Perforce.
With Perforce, you get hassle-free workflows. Merge that actually works. 
Faster operations. Version large binaries.  Built-in WAN optimization and the
freedom to use Git, Perforce or both. Make the move to Perforce.
http://pubads.g.doubleclick.net/gampad/clk?id=122218951&iu=/4140/ostg.clktrk
_______________________________________________
tmux-cvs mailing list
tmux-cvs@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/tmux-cvs

Reply via email to