Tag: use new[]/delete[] instead of g_new()/g_free()
This commit is contained in:
		| @@ -25,7 +25,6 @@ | ||||
| #include "TagBuilder.hxx" | ||||
| #include "util/ASCII.hxx" | ||||
|  | ||||
| #include <glib.h> | ||||
| #include <assert.h> | ||||
| #include <string.h> | ||||
|  | ||||
| @@ -59,12 +58,6 @@ tag_name_parse_i(const char *name) | ||||
| 	return TAG_NUM_OF_ITEM_TYPES; | ||||
| } | ||||
|  | ||||
| static size_t | ||||
| items_size(const Tag &tag) | ||||
| { | ||||
| 	return tag.num_items * sizeof(TagItem *); | ||||
| } | ||||
|  | ||||
| void | ||||
| Tag::Clear() | ||||
| { | ||||
| @@ -76,7 +69,7 @@ Tag::Clear() | ||||
| 		tag_pool_put_item(items[i]); | ||||
| 	tag_pool_lock.unlock(); | ||||
|  | ||||
| 	g_free(items); | ||||
| 	delete[] items; | ||||
| 	items = nullptr; | ||||
| 	num_items = 0; | ||||
| } | ||||
| @@ -88,7 +81,7 @@ Tag::~Tag() | ||||
| 		tag_pool_put_item(items[i]); | ||||
| 	tag_pool_lock.unlock(); | ||||
|  | ||||
| 	g_free(items); | ||||
| 	delete[] items; | ||||
| } | ||||
|  | ||||
| Tag::Tag(const Tag &other) | ||||
| @@ -97,7 +90,7 @@ Tag::Tag(const Tag &other) | ||||
| 	 num_items(other.num_items) | ||||
| { | ||||
| 	if (num_items > 0) { | ||||
| 		items = (TagItem **)g_malloc(items_size(other)); | ||||
| 		items = new TagItem *[num_items]; | ||||
|  | ||||
| 		tag_pool_lock.lock(); | ||||
| 		for (unsigned i = 0; i < num_items; i++) | ||||
|   | ||||
| @@ -125,7 +125,7 @@ TagBuilder::Commit(Tag &tag) | ||||
| 	   object */ | ||||
| 	const unsigned n_items = items.size(); | ||||
| 	tag.num_items = n_items; | ||||
| 	tag.items = g_new(TagItem *, n_items); | ||||
| 	tag.items = new TagItem *[n_items]; | ||||
| 	std::copy_n(items.begin(), n_items, tag.items); | ||||
| 	items.clear(); | ||||
|  | ||||
|   | ||||
		Reference in New Issue
	
	Block a user
	 Max Kellermann
					Max Kellermann