Em Sat, Apr 30, 2016 at 12:11:00AM +0900, Masami Hiramatsu escreveu:
> Remove unused xrealloc() and ALLOC_GROW() from libperf.
Whee! Thank you!
- Arnaldo
> Signed-off-by: Masami Hiramatsu <mhira...@kernel.org>
> ---
> tools/perf/util/Build | 1 -
> tools/perf/util/cache.h | 19 -------------------
> tools/perf/util/util.h | 6 ------
> tools/perf/util/wrapper.c | 29 -----------------------------
> 4 files changed, 55 deletions(-)
> delete mode 100644 tools/perf/util/wrapper.c
>
> diff --git a/tools/perf/util/Build b/tools/perf/util/Build
> index 90229a8..b7768ea 100644
> --- a/tools/perf/util/Build
> +++ b/tools/perf/util/Build
> @@ -27,7 +27,6 @@ libperf-y += strlist.o
> libperf-y += strfilter.o
> libperf-y += top.o
> libperf-y += usage.o
> -libperf-y += wrapper.o
> libperf-y += dso.o
> libperf-y += symbol.o
> libperf-y += symbol_fprintf.o
> diff --git a/tools/perf/util/cache.h b/tools/perf/util/cache.h
> index 1f5a93c..0d814bb 100644
> --- a/tools/perf/util/cache.h
> +++ b/tools/perf/util/cache.h
> @@ -40,25 +40,6 @@ int split_cmdline(char *cmdline, const char ***argv);
>
> #define alloc_nr(x) (((x)+16)*3/2)
>
> -/*
> - * Realloc the buffer pointed at by variable 'x' so that it can hold
> - * at least 'nr' entries; the number of entries currently allocated
> - * is 'alloc', using the standard growing factor alloc_nr() macro.
> - *
> - * DO NOT USE any expression with side-effect for 'x' or 'alloc'.
> - */
> -#define ALLOC_GROW(x, nr, alloc) \
> - do { \
> - if ((nr) > alloc) { \
> - if (alloc_nr(alloc) < (nr)) \
> - alloc = (nr); \
> - else \
> - alloc = alloc_nr(alloc); \
> - x = xrealloc((x), alloc * sizeof(*(x))); \
> - } \
> - } while(0)
> -
> -
> static inline int is_absolute_path(const char *path)
> {
> return path[0] == '/';
> diff --git a/tools/perf/util/util.h b/tools/perf/util/util.h
> index 88f607a..7651633 100644
> --- a/tools/perf/util/util.h
> +++ b/tools/perf/util/util.h
> @@ -160,12 +160,6 @@ static inline char *gitstrchrnul(const char *s, int c)
> }
> #endif
>
> -/*
> - * Wrappers:
> - */
> -void *xrealloc(void *ptr, size_t size) __attribute__((weak));
> -
> -
> static inline void *zalloc(size_t size)
> {
> return calloc(1, size);
> diff --git a/tools/perf/util/wrapper.c b/tools/perf/util/wrapper.c
> deleted file mode 100644
> index 5f1a07c..0000000
> --- a/tools/perf/util/wrapper.c
> +++ /dev/null
> @@ -1,29 +0,0 @@
> -/*
> - * Various trivial helper wrappers around standard functions
> - */
> -#include "cache.h"
> -
> -/*
> - * There's no pack memory to release - but stay close to the Git
> - * version so wrap this away:
> - */
> -static inline void release_pack_memory(size_t size __maybe_unused,
> - int flag __maybe_unused)
> -{
> -}
> -
> -void *xrealloc(void *ptr, size_t size)
> -{
> - void *ret = realloc(ptr, size);
> - if (!ret && !size)
> - ret = realloc(ptr, 1);
> - if (!ret) {
> - release_pack_memory(size, -1);
> - ret = realloc(ptr, size);
> - if (!ret && !size)
> - ret = realloc(ptr, 1);
> - if (!ret)
> - die("Out of memory, realloc failed");
> - }
> - return ret;
> -}
