[pulseaudio-discuss] [PATCH 1/2] pulsecore: Don't conditionally inline pa_run_from_build_tree()

Arun Raghavan arun.raghavan at collabora.co.uk
Sat Apr 13 23:08:20 PDT 2013


There's no good reason to assume an in-tree build will be debug-only.
This breaks alsa-mixer-path-test on make distcheck, for example.
---
 src/pulsecore/core-util.c | 4 ----
 src/pulsecore/core-util.h | 9 +--------
 2 files changed, 1 insertion(+), 12 deletions(-)

diff --git a/src/pulsecore/core-util.c b/src/pulsecore/core-util.c
index 1f4fca1..657e500 100644
--- a/src/pulsecore/core-util.c
+++ b/src/pulsecore/core-util.c
@@ -3207,8 +3207,6 @@ void pa_reset_personality(void) {
 
 }
 
-#if defined(__linux__) && !defined(__OPTIMIZE__)
-
 pa_bool_t pa_run_from_build_tree(void) {
     char *rp;
     pa_bool_t b = FALSE;
@@ -3221,8 +3219,6 @@ pa_bool_t pa_run_from_build_tree(void) {
     return b;
 }
 
-#endif
-
 const char *pa_get_temp_dir(void) {
     const char *t;
 
diff --git a/src/pulsecore/core-util.h b/src/pulsecore/core-util.h
index 9d59383..15e6809 100644
--- a/src/pulsecore/core-util.h
+++ b/src/pulsecore/core-util.h
@@ -260,14 +260,7 @@ size_t pa_pipe_buf(int fd);
 
 void pa_reset_personality(void);
 
-/* We abuse __OPTIMIZE__ as a check whether we are a debug build
- * or not. If we are and are run from the build tree then we
- * override the search path to point to our build tree */
-#if defined(__linux__) && !defined(__OPTIMIZE__)
-pa_bool_t pa_run_from_build_tree(void);
-#else
-static inline pa_bool_t pa_run_from_build_tree(void) {return FALSE;}
-#endif
+pa_bool_t pa_run_from_build_tree(void) PA_GCC_CONST;
 
 const char *pa_get_temp_dir(void);
 
-- 
1.8.1.2



More information about the pulseaudio-discuss mailing list