summaryrefslogtreecommitdiffstats
path: root/src/Structure.cpp
diff options
context:
space:
mode:
authorGravatar Rosen Penev2020-12-21 22:09:42 -0800
committerGravatar Nikias Bassen2021-06-22 01:29:05 +0200
commitb086d0612740978c1942c1c3a078d4a0b8ffffef (patch)
tree46484950e8e6dbf2fb5d773a84b5bb73fc313bf7 /src/Structure.cpp
parent526c10436caa894fb6139f4e6c0f73eebfa858ea (diff)
downloadlibplist-b086d0612740978c1942c1c3a078d4a0b8ffffef.tar.gz
libplist-b086d0612740978c1942c1c3a078d4a0b8ffffef.tar.bz2
[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 <rosenp@gmail.com>
Diffstat (limited to 'src/Structure.cpp')
-rw-r--r--src/Structure.cpp4
1 files changed, 2 insertions, 2 deletions
diff --git a/src/Structure.cpp b/src/Structure.cpp
index 9445c23..4be4e7d 100644
--- a/src/Structure.cpp
+++ b/src/Structure.cpp
@@ -56,7 +56,7 @@ std::string Structure::ToXml() const
56 uint32_t length = 0; 56 uint32_t length = 0;
57 plist_to_xml(_node, &xml, &length); 57 plist_to_xml(_node, &xml, &length);
58 std::string ret(xml, xml+length); 58 std::string ret(xml, xml+length);
59 free(xml); 59 delete xml;
60 return ret; 60 return ret;
61} 61}
62 62
@@ -66,7 +66,7 @@ std::vector<char> Structure::ToBin() const
66 uint32_t length = 0; 66 uint32_t length = 0;
67 plist_to_bin(_node, &bin, &length); 67 plist_to_bin(_node, &bin, &length);
68 std::vector<char> ret(bin, bin+length); 68 std::vector<char> ret(bin, bin+length);
69 free(bin); 69 delete bin;
70 return ret; 70 return ret;
71} 71}
72 72