From b086d0612740978c1942c1c3a078d4a0b8ffffef Mon Sep 17 00:00:00 2001 From: Rosen Penev Date: Mon, 21 Dec 2020 22:09:42 -0800 Subject: [clang-tidy] cpp: Replace free with delete It's the C++ way. It also avoids having to check for NULL. Found with cppcoreguidelines-owning-memory Signed-off-by: Rosen Penev --- src/Dictionary.cpp | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) (limited to 'src/Dictionary.cpp') diff --git a/src/Dictionary.cpp b/src/Dictionary.cpp index 20e9710..ea04e81 100644 --- a/src/Dictionary.cpp +++ b/src/Dictionary.cpp @@ -39,7 +39,7 @@ static void dictionary_fill(Dictionary *_this, std::map &map, plist_dict_next_item(node, it, &key, &subnode); if (key && subnode) map[std::string(key)] = Node::FromPlist(subnode, _this); - free(key); + delete key; } while (subnode); free(it); } @@ -156,7 +156,7 @@ void Dictionary::Remove(Node* node) plist_dict_get_item_key(node->GetPlist(), &key); plist_dict_remove_item(_node, key); std::string skey = key; - free(key); + delete key; _map.erase(skey); delete node; } -- cgit v1.1-32-gdbae