Commit d6941503 authored by Max Kellermann's avatar Max Kellermann

Tag: destructor calls Clear()

Eliminate duplicate code.
parent 4ab586aa
...@@ -74,16 +74,6 @@ Tag::Clear() ...@@ -74,16 +74,6 @@ Tag::Clear()
num_items = 0; num_items = 0;
} }
Tag::~Tag()
{
tag_pool_lock.lock();
for (int i = num_items; --i >= 0; )
tag_pool_put_item(items[i]);
tag_pool_lock.unlock();
delete[] items;
}
Tag::Tag(const Tag &other) Tag::Tag(const Tag &other)
:time(other.time), has_playlist(other.has_playlist), :time(other.time), has_playlist(other.has_playlist),
items(nullptr), items(nullptr),
......
...@@ -71,7 +71,9 @@ struct Tag { ...@@ -71,7 +71,9 @@ struct Tag {
/** /**
* Free the tag object and all its items. * Free the tag object and all its items.
*/ */
~Tag(); ~Tag() {
Clear();
}
Tag &operator=(const Tag &other) = delete; Tag &operator=(const Tag &other) = delete;
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment