From: Simon Ruderich <simon@ruderich.org>
Date: Sat, 1 Jun 2013 20:44:22 +0000 (+0200)
Subject: hookmacros.h: Reduce duplication in HOOK_VOID1().
X-Git-Tag: 0.1~126
X-Git-Url: https://ruderich.org/simon/gitweb/?a=commitdiff_plain;h=73870d5ed5fc275c462eefff724e3b26502ce26a;p=coloredstderr%2Fcoloredstderr.git

hookmacros.h: Reduce duplication in HOOK_VOID1().
---

diff --git a/src/hookmacros.h b/src/hookmacros.h
index 9e1bd75..09e718a 100644
--- a/src/hookmacros.h
+++ b/src/hookmacros.h
@@ -26,17 +26,18 @@
 
 #define _HOOK_PRE(type, name) \
         int handle; \
-        type result; \
         DLSYM_FUNCTION(real_ ## name, #name);
 #define _HOOK_PRE_FD(type, name, fd) \
+        type result; \
+        _HOOK_PRE_FD_(type, name, fd)
+#define _HOOK_PRE_FD_(type, name, fd) \
         _HOOK_PRE(type, name) \
-        _HOOK_PRE_FD_ALONE(type, name, fd)
-#define _HOOK_PRE_FD_ALONE(type, name, fd) \
         handle = check_handle_fd(fd); \
         if (handle) { \
             handle_fd_pre(fd, handle); \
         }
 #define _HOOK_PRE_FILE(type, name, file) \
+        type result; \
         _HOOK_PRE(type, name) \
         handle = check_handle_fd(fileno(file)); \
         if (handle) { \
@@ -44,14 +45,14 @@
         }
 /* Save and restore the errno to make sure we return the errno of the original
  * function call. */
-#define _HOOK_POST_FD_ALONE(fd) \
+#define _HOOK_POST_FD_(fd) \
         if (handle) { \
             int saved_errno = errno; \
             handle_fd_post(fd, handle); \
             errno = saved_errno; \
         }
 #define _HOOK_POST_FD(fd) \
-        _HOOK_POST_FD_ALONE(fd) \
+        _HOOK_POST_FD_(fd) \
         return result;
 #define _HOOK_POST_FILE(file) \
         if (handle) { \
@@ -65,11 +66,9 @@
 #define HOOK_VOID1(type, name, fd, type1, arg1) \
     static type (*real_ ## name)(type1); \
     type name(type1 arg1) { \
-        int handle; \
-        DLSYM_FUNCTION(real_ ## name, #name); \
-        _HOOK_PRE_FD_ALONE(type, name, fd) \
+        _HOOK_PRE_FD_(type, name, fd) \
         real_ ## name(arg1); \
-        _HOOK_POST_FD_ALONE(fd) \
+        _HOOK_POST_FD_(fd) \
     }
 
 #define HOOK_FD3(type, name, fd, type1, arg1, type2, arg2, type3, arg3) \