From 9c4f0d3a6e36f8901e3a85be6269b9ec652b8eea Mon Sep 17 00:00:00 2001 From: Eric Wong Date: Sun, 28 Sep 2008 00:36:03 -0700 Subject: songvec_free => songvec_destroy "free" implies the songvec structure itself is freed, which is not the case. --- src/songvec.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) (limited to 'src/songvec.c') diff --git a/src/songvec.c b/src/songvec.c index 842e7afcd..7ce1285f6 100644 --- a/src/songvec.c +++ b/src/songvec.c @@ -59,7 +59,7 @@ void songvec_add(struct songvec *sv, Song *add) sv->base[sv->nr - 1] = add; } -void songvec_free(struct songvec *sv) +void songvec_destroy(struct songvec *sv) { if (sv->base) { free(sv->base); -- cgit v1.2.3 From 9fb0f4f7cc3e4b2e1fb203a1949799a6018fb57b Mon Sep 17 00:00:00 2001 From: Eric Wong Date: Sun, 28 Sep 2008 19:17:00 -0700 Subject: songvec: songvec_delete takes a const Song pointer We don't modify the Song when we delete it --- src/songvec.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) (limited to 'src/songvec.c') diff --git a/src/songvec.c b/src/songvec.c index 7ce1285f6..579ac7033 100644 --- a/src/songvec.c +++ b/src/songvec.c @@ -30,7 +30,7 @@ Song *songvec_find(struct songvec *sv, const char *url) return NULL; } -int songvec_delete(struct songvec *sv, Song *del) +int songvec_delete(struct songvec *sv, const Song *del) { int i; -- cgit v1.2.3