tag: merge clearMpdTag into tag_free
avoid some redundant clearing logic as well, since the tag is getting freed.
This commit is contained in:
parent
8faf648784
commit
fb4d55c5b3
14
src/tag.c
14
src/tag.c
@ -273,12 +273,12 @@ void tag_clear_items_by_type(struct tag *tag, enum tag_type type)
|
|||||||
}
|
}
|
||||||
}
|
}
|
||||||
|
|
||||||
static void clearMpdTag(struct tag *tag)
|
void tag_free(struct tag *tag)
|
||||||
{
|
{
|
||||||
int i;
|
int i;
|
||||||
|
|
||||||
pthread_mutex_lock(&tag_pool_lock);
|
pthread_mutex_lock(&tag_pool_lock);
|
||||||
for (i = 0; i < tag->numOfItems; i++)
|
for (i = tag->numOfItems; --i >= 0; )
|
||||||
tag_pool_put_item(tag->items[i]);
|
tag_pool_put_item(tag->items[i]);
|
||||||
pthread_mutex_unlock(&tag_pool_lock);
|
pthread_mutex_unlock(&tag_pool_lock);
|
||||||
|
|
||||||
@ -291,16 +291,6 @@ static void clearMpdTag(struct tag *tag)
|
|||||||
free(tag->items);
|
free(tag->items);
|
||||||
}
|
}
|
||||||
|
|
||||||
tag->items = NULL;
|
|
||||||
|
|
||||||
tag->numOfItems = 0;
|
|
||||||
|
|
||||||
tag->time = -1;
|
|
||||||
}
|
|
||||||
|
|
||||||
void tag_free(struct tag *tag)
|
|
||||||
{
|
|
||||||
clearMpdTag(tag);
|
|
||||||
free(tag);
|
free(tag);
|
||||||
}
|
}
|
||||||
|
|
||||||
|
Loading…
x
Reference in New Issue
Block a user