configure.ac: Add --enable-warnings.

Enables warning() messages without the overhead of all debug output.

diff --git a/configure.ac b/configure.ac
index e17afdf7dbf41d4265fb4d2b9159c402a037e54a..f4154bd04a9b25694b1ed59fac63d4ebe9becc33 100644 (file)
--- a/configure.ac
+++ b/configure.ac
@@ -58,10 +58,17 @@ AC_LINK_IFELSE([AC_LANG_PROGRAM([],[return __builtin_expect(main != 0, 1)])],
 
 AC_SEARCH_LIBS([dlsym], [dl], [], [AC_MSG_ERROR([dlsym() is required])])
 
+AC_ARG_ENABLE([warnings],
+              [AS_HELP_STRING([--enable-warnings],[enable warning output])],
+              [if test "x$enableval" = xyes; then
+                   AC_DEFINE([WARNING], 1, [Define to enable warning output.])
+               fi])
 AC_ARG_ENABLE([debug],
               [AS_HELP_STRING([--enable-debug],[enable debug output])],
               [if test "x$enableval" = xyes; then
                    AC_DEFINE([DEBUG], 1, [Define to enable debug output.])
+                   dnl DEBUG implies WARNING
+                   AC_DEFINE([WARNING], 1)
                fi])
 
 dnl Used in tests/Makefile.am to build the test only if error() is available.

diff --git a/src/coloredstderr.c b/src/coloredstderr.c
index dcd1166edae9f86d65f8d57c57cee2f7096ccfef..d0c5494e27d27308b35096d418c032f3c1e2f143 100644 (file)
--- a/src/coloredstderr.c
+++ b/src/coloredstderr.c
@@ -53,7 +53,7 @@ static int force_write_to_non_tty;
 
 
 #include "constants.h"
-#ifdef DEBUG
+#ifdef WARNING
 # include "debug.h"
 #endif
 

diff --git a/src/constants.h b/src/constants.h
index f76f78f490e0a1d21dd64734cd3dc8b80daba0de..bde43b5a90b8df9d8b998b1f555d57e6f5e896a8 100644 (file)
--- a/src/constants.h
+++ b/src/constants.h
@@ -39,6 +39,8 @@
 
 #ifdef DEBUG
 # define DEBUG_FILE "colored_stderr_debug_log.txt"
+#endif
+#ifdef WARNING
 /* Created in the user's home directory, appends to existing file. */
 # define WARNING_FILE "colored_stderr_warning_log.txt"
 #endif

diff --git a/src/debug.h b/src/debug.h
index 7425ce475f8c49f475f4dba01d48408ee65f547b..4a7c00bc107949f2a9ab5b7aaea3c3c1452a429e 100644 (file)
--- a/src/debug.h
+++ b/src/debug.h
@@ -41,6 +41,7 @@ static void debug_write(int fd, int first_call, char const *format, va_list ap)
     real_close(fd);
 }
 
+#ifdef DEBUG
 static void debug(char const *format, ...) {
     va_list ap;
 
@@ -63,6 +64,7 @@ static void debug(char const *format, ...) {
 
     errno = saved_errno;
 }
+#endif
 
 static void warning(char const *format, ...) {
     va_list ap;

diff --git a/src/trackfds.h b/src/trackfds.h
index 2def68cfb99dc4233d9df7c74667108966cd1b30..1eede800ddcd745f7e8eb1a4595157260bf74131 100644 (file)
--- a/src/trackfds.h
+++ b/src/trackfds.h
@@ -56,7 +56,7 @@ static int init_tracked_fds_list(size_t count) {
 
     tracked_fds_list = malloc(count * sizeof(*tracked_fds_list));
     if (!tracked_fds_list) {
-#ifdef DEBUG
+#ifdef WARNING
         warning("malloc(tracked_fds_list, %d) failed [%d]\n",
                 count * sizeof(*tracked_fds_list), getpid());
 #endif
@@ -164,7 +164,7 @@ static char *update_environment_buffer_entry(char *x, int fd) {
     int length = snprintf(x, 10 + 1, "%d", fd);
     if (length >= 10 + 1) {
         /* Integer too big to fit the buffer, skip it. */
-#ifdef DEBUG
+#ifdef WARNING
         warning("update_environment_buffer_entry(): truncated fd: %d [%d]\n",
                 fd, getpid());
 #endif
@@ -245,7 +245,7 @@ static void tracked_fds_add(int fd) {
             /* We can do nothing, just ignore the error. We made sure not to
              * destroy our state, so the new descriptor is ignored without any
              * other consequences. */
-#ifdef DEBUG
+#ifdef WARNING
             warning("realloc(tracked_fds_list, %zu) failed! [%d]\n",
                     sizeof(*tracked_fds_list) * new_space, getpid());
 #endif