aboutsummaryrefslogtreecommitdiff
diff options
context:
space:
mode:
-rw-r--r--src/decoder/wavpack_plugin.c2
-rw-r--r--src/event_pipe.c3
-rw-r--r--src/utils.c58
-rw-r--r--src/utils.h57
4 files changed, 8 insertions, 112 deletions
diff --git a/src/decoder/wavpack_plugin.c b/src/decoder/wavpack_plugin.c
index b2100f09..11b85622 100644
--- a/src/decoder/wavpack_plugin.c
+++ b/src/decoder/wavpack_plugin.c
@@ -27,6 +27,8 @@
#include <assert.h>
#include <unistd.h>
+#include <stdio.h>
+#include <stdlib.h>
/* pick 1020 since its devisible for 8,16,24, and 32-bit audio */
#define CHUNK_SIZE 1020
diff --git a/src/event_pipe.c b/src/event_pipe.c
index 87e40c4e..0a2debff 100644
--- a/src/event_pipe.c
+++ b/src/event_pipe.c
@@ -25,6 +25,9 @@
#include <assert.h>
#include <glib.h>
#include <string.h>
+#include <errno.h>
+#include <sys/types.h>
+#include <unistd.h>
#ifdef WIN32
/* for _O_BINARY */
diff --git a/src/utils.c b/src/utils.c
index c3071266..43ccb2fc 100644
--- a/src/utils.c
+++ b/src/utils.c
@@ -27,6 +27,7 @@
#include <string.h>
#include <sys/types.h>
#include <fcntl.h>
+#include <errno.h>
#ifndef WIN32
#include <pwd.h>
@@ -54,55 +55,6 @@ void my_usleep(long usec)
#endif
}
-G_GNUC_MALLOC char *xstrdup(const char *s)
-{
- char *ret = strdup(s);
- if (G_UNLIKELY(!ret))
- g_error("OOM: strdup");
- return ret;
-}
-
-/* borrowed from git :) */
-
-G_GNUC_MALLOC void *xmalloc(size_t size)
-{
- void *ret;
-
- assert(G_LIKELY(size));
-
- ret = malloc(size);
- if (G_UNLIKELY(!ret))
- g_error("OOM: malloc");
- return ret;
-}
-
-G_GNUC_MALLOC void *xrealloc(void *ptr, size_t size)
-{
- void *ret = realloc(ptr, size);
-
- /* some C libraries return NULL when size == 0,
- * make sure we get a free()-able pointer (free(NULL)
- * doesn't work with all C libraries, either) */
- if (G_UNLIKELY(!ret && !size))
- ret = realloc(ptr, 1);
-
- if (G_UNLIKELY(!ret))
- g_error("OOM: realloc");
- return ret;
-}
-
-G_GNUC_MALLOC void *xcalloc(size_t nmemb, size_t size)
-{
- void *ret;
-
- assert(G_LIKELY(nmemb && size));
-
- ret = calloc(nmemb, size);
- if (G_UNLIKELY(!ret))
- g_error("OOM: calloc");
- return ret;
-}
-
char *parsePath(char *path)
{
#ifndef WIN32
@@ -112,7 +64,6 @@ char *parsePath(char *path)
} else if (path[0] == '~') {
size_t pos = 1;
const char *home;
- char *newPath;
if (path[1] == '/' || path[1] == '\0') {
ConfigParam *param = getConfigParam(CONF_USER);
@@ -157,13 +108,10 @@ char *parsePath(char *path)
home = passwd->pw_dir;
}
- newPath = xmalloc(strlen(home) + strlen(path + pos) + 1);
- strcpy(newPath, home);
- strcat(newPath, path + pos);
- return newPath;
+ return g_strconcat(home, path + pos, NULL);
} else {
#endif
- return xstrdup(path);
+ return g_strdup(path);
#ifndef WIN32
}
#endif
diff --git a/src/utils.h b/src/utils.h
index dff41bab..d3fb7a61 100644
--- a/src/utils.h
+++ b/src/utils.h
@@ -19,14 +19,6 @@
#ifndef MPD_UTILS_H
#define MPD_UTILS_H
-#include <glib.h>
-#include <unistd.h>
-#include <stdlib.h>
-#include <stdio.h>
-#include <errno.h>
-
-#define ARRAY_SIZE(x) (sizeof(x)/sizeof(x[0]))
-
#ifndef assert_static
/* Compile time assertion developed by Ralf Holly */
/* http://pera-software.com/articles/compile-time-assertions.pdf */
@@ -38,55 +30,6 @@
void my_usleep(long usec);
-/* trivial functions, keep them inlined */
-static inline void xclose(int fd)
-{
- while (close(fd) && errno == EINTR);
-}
-
-static inline ssize_t xread(int fd, void *buf, size_t len)
-{
- ssize_t nr;
- while (1) {
- nr = read(fd, buf, len);
- if ((nr < 0) && (errno == EAGAIN || errno == EINTR))
- continue;
- return nr;
- }
-}
-
-static inline ssize_t xwrite(int fd, const void *buf, size_t len)
-{
- ssize_t nr;
- while (1) {
- nr = write(fd, buf, len);
- if ((nr < 0) && (errno == EAGAIN || errno == EINTR))
- continue;
- return nr;
- }
-}
-
-G_GNUC_MALLOC char *xstrdup(const char *s);
-
-G_GNUC_MALLOC void *xmalloc(size_t size);
-
-G_GNUC_MALLOC void *xrealloc(void *ptr, size_t size);
-
-G_GNUC_MALLOC void *xcalloc(size_t nmemb, size_t size);
-
-/**
- * free a const pointer - unfortunately free() expects a non-const
- * pointer, for whatever reason
- */
-static inline void xfree(const void *p)
-{
- union {
- const void *in;
- void *out;
- } deconst = { .in = p };
- free(deconst.out);
-}
-
char *parsePath(char *path);
int set_nonblocking(int fd);