X-Git-Url: https://ruderich.org/simon/gitweb/?a=blobdiff_plain;f=src%2Fcoloredstderr.c;h=dcd1166edae9f86d65f8d57c57cee2f7096ccfef;hb=b75b0ec399b1217568e08e567afdefaea0bd32ef;hp=72a881fb866ab12e0e92188c58456e4e2979c1a7;hpb=048bd6f9e7f316aafede310ba273776d0086b8ed;p=coloredstderr%2Fcoloredstderr.git diff --git a/src/coloredstderr.c b/src/coloredstderr.c index 72a881f..dcd1166 100644 --- a/src/coloredstderr.c +++ b/src/coloredstderr.c @@ -20,6 +20,8 @@ #include +#include "compiler.h" + /* Must be loaded before the following headers. */ #include "ldpreload.h" @@ -51,7 +53,6 @@ static int force_write_to_non_tty; #include "constants.h" -#include "compiler.h" #ifdef DEBUG # include "debug.h" #endif @@ -64,6 +65,19 @@ static int force_write_to_non_tty; /* See hookmacros.h for the decision if a function call is colored. */ +/* Prevent inlining into hook functions because it may increase the number of + * spilled registers unnecessarily. As it's not called very often accept the + * additional call. */ +static int isatty_noinline(int fd) noinline; +static int isatty_noinline(int fd) { + int saved_errno = errno; + int result = isatty(fd); + errno = saved_errno; + + return result; +} + + static void dup_fd(int oldfd, int newfd) { #ifdef DEBUG debug("%3d -> %3d\t\t\t[%d]\n", oldfd, newfd, getpid()); @@ -108,7 +122,7 @@ static size_t post_string_size; /* Load alternative pre/post strings from the environment if available, fall * back to default values. */ -inline static void init_pre_post_string(void) { +static void init_pre_post_string(void) { pre_string = getenv(ENV_NAME_PRE_STRING); if (!pre_string) { pre_string = DEFAULT_PRE_STRING; @@ -122,6 +136,13 @@ inline static void init_pre_post_string(void) { post_string_size = strlen(post_string); } +/* Don't inline any of the pre/post functions. Keep the hook function as small + * as possible for speed reasons. */ +static void handle_fd_pre(int fd) noinline; +static void handle_fd_post(int fd) noinline; +static void handle_file_pre(FILE *stream) noinline; +static void handle_file_post(FILE *stream) noinline; + static void handle_fd_pre(int fd) { int saved_errno = errno;