diff options
author | Eric Wong <normalperson@yhbt.net> | 2008-10-13 21:54:02 -0700 |
---|---|---|
committer | Eric Wong <normalperson@yhbt.net> | 2008-10-13 21:54:02 -0700 |
commit | 396f03db551d585ee3eff4a3a1e6f49283abbc61 (patch) | |
tree | e5566191b76c19e50222e41d7a835e0652a9b2be /src/songvec.c | |
parent | e5c604feac962f654ba2cb7739ef33b9242cb072 (diff) | |
download | mpd-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 'src/songvec.c')
-rw-r--r-- | src/songvec.c | 11 |
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) { |