aboutsummaryrefslogtreecommitdiffstats
path: root/src/songvec.c
diff options
context:
space:
mode:
authorEric Wong <normalperson@yhbt.net>2008-10-13 21:54:02 -0700
committerEric Wong <normalperson@yhbt.net>2008-10-13 21:54:02 -0700
commit396f03db551d585ee3eff4a3a1e6f49283abbc61 (patch)
treee5566191b76c19e50222e41d7a835e0652a9b2be /src/songvec.c
parente5c604feac962f654ba2cb7739ef33b9242cb072 (diff)
downloadmpd-396f03db551d585ee3eff4a3a1e6f49283abbc61.tar.gz
mpd-396f03db551d585ee3eff4a3a1e6f49283abbc61.tar.xz
mpd-396f03db551d585ee3eff4a3a1e6f49283abbc61.zip
{dir,song}vec: remove unused *_destroy routines
Blindly destroying these without freeing the underlying elements was a bad idea, always. Our iterators suck less nowadays and we can traverse them and free() each element safely.
Diffstat (limited to '')
-rw-r--r--src/songvec.c11
1 files changed, 0 insertions, 11 deletions
diff --git a/src/songvec.c b/src/songvec.c
index 613c448cf..dfab51451 100644
--- a/src/songvec.c
+++ b/src/songvec.c
@@ -73,17 +73,6 @@ void songvec_add(struct songvec *sv, struct mpd_song *add)
pthread_mutex_unlock(&nr_lock);
}
-void songvec_destroy(struct songvec *sv)
-{
- pthread_mutex_lock(&nr_lock);
- sv->nr = 0;
- pthread_mutex_unlock(&nr_lock);
- if (sv->base) {
- free(sv->base);
- sv->base = NULL;
- }
-}
-
int songvec_for_each(const struct songvec *sv,
int (*fn)(struct mpd_song *, void *), void *arg)
{