ecore_audio: uncondionally include libgen.h.
authorBenjamin Jacobs <benj@spam.thsi.be>
Mon, 6 Jun 2016 18:18:25 +0000 (11:18 -0700)
committerCedric BAIL <cedric@osg.samsung.com>
Mon, 6 Jun 2016 19:05:43 +0000 (12:05 -0700)
Basically everyone should have it.  If not, maybe we should consider
to test the value of _POSIX_C_SOURCE before testing platform specific
defines, or adding a feature check.

Fixes the following warning on DragonFlyBSD:

lib/ecore_audio/ecore_audio_obj_out_pulse.c:282:60: warning: implicit declaration of function 'basename' [-Wimplicit-function-declaration]
        class_vars.context = pa_context_new(class_vars.api, basename(argv[0]));
                                                            ^
lib/ecore_audio/ecore_audio_obj_out_pulse.c:282:60: warning: passing argument 2 of 'pa_context_new' makes pointer from integer without a cast [-Wint-conversion]
In file included from /usr/local/include/pulse/pulseaudio.h:29:0,
                 from lib/ecore_audio/ecore_audio_private.h:17,
                 from lib/ecore_audio/ecore_audio_obj_out_pulse.c:18:
/usr/local/include/pulse/context.h:172:13: note: expected 'const char *' but argument is of type 'int'
 pa_context *pa_context_new(pa_mainloop_api *mainloop, const char *name);
             ^

Signed-off-by: Cedric BAIL <cedric@osg.samsung.com>
src/lib/ecore_audio/ecore_audio_obj_out_pulse.c

index 9f2961e..05c42b4 100644 (file)
@@ -6,9 +6,7 @@
 #include <stdio.h>
 #include <string.h>
 
-#if defined (__MacOSX__) || (defined (__MACH__) && defined (__APPLE__)) || defined (__FreeBSD__)
-# include <libgen.h>
-#endif
+#include <libgen.h>
 
 #ifdef HAVE_FEATURES_H
 #include <features.h>