refactor: make iniConf store as shared_ptr

Author: ShrBox

src/legacy/api/DataAPI.cpp

@@ -269,13 +269,13 @@ bool ConfJsonClass::reload() {
 ConfIniClass::ConfIniClass(const Local<Object>& scriptObj, const string& path, const string& defContent)
 : ScriptClass(scriptObj),
   ConfBaseClass(path) {
-    iniConf = SimpleIni::create(path, defContent).release();
+    iniConf = SimpleIni::create(path, defContent);
 }
 
 ConfIniClass::ConfIniClass(const string& path, const string& defContent)
 : ScriptClass(ScriptClass::ConstructFromCpp<ConfIniClass>{}),
   ConfBaseClass(path) {
-    iniConf = SimpleIni::create(path, defContent).release();
+    iniConf = SimpleIni::create(path, defContent);
 }
 
 ConfIniClass::~ConfIniClass() { close(); }
@@ -300,16 +300,15 @@ bool ConfIniClass::flush() { return iniConf->SaveFile(iniConf->filePath.c_str(),
 bool ConfIniClass::close() {
     if (isValid()) {
         reload();
-        delete iniConf;
-        iniConf = nullptr;
+        iniConf.reset();
     }
     return true;
 }
 bool ConfIniClass::reload() {
     if (!isValid()) return false;
 
-    delete iniConf;
-    iniConf = SimpleIni::create(confPath).release();
+    iniConf.reset();
+    iniConf = SimpleIni::create(confPath);
     return true;
 }
 

src/legacy/api/DataAPI.h

@@ -97,10 +97,10 @@ extern ClassDefine<ConfJsonClass> ConfJsonClassBuilder;
 
 class ConfIniClass : public ScriptClass, public ConfBaseClass {
 private:
-    SimpleIni* iniConf;
-    bool       flush() override;
-    bool       close() override;
-    bool       reload() override;
+    std::shared_ptr<SimpleIni> iniConf;
+    bool                       flush() override;
+    bool                       close() override;
+    bool                       reload() override;
 
 public:
     explicit ConfIniClass(const Local<Object>& scriptObj, const std::string& path, const std::string& defContent);

src/legacy/utils/IniHelper.cpp

@@ -5,7 +5,7 @@
 
 #include <filesystem>
 
-std::unique_ptr<SimpleIni> SimpleIni::create(const std::string& path, const std::string& defContent) {
+std::shared_ptr<SimpleIni> SimpleIni::create(const std::string& path, const std::string& defContent) {
     namespace fs = std::filesystem;
     auto fpath   = fs::path(ll::string_utils::str2wstr(path));
     if (!fpath.empty() && !fs::exists(fpath)) { // Create new file
@@ -18,7 +18,7 @@ std::unique_ptr<SimpleIni> SimpleIni::create(const std::string& path, const std:
     }
 
     // Exist
-    auto root = std::make_unique<SimpleIni>();
+    auto root = std::make_shared<SimpleIni>();
     root->SetUnicode(true);
     auto res = root->LoadFile(path.c_str());
     if (res < 0) {

src/legacy/utils/IniHelper.h

@@ -8,8 +8,8 @@ class SimpleIni : public CSimpleIniA {
 public:
     std::string filePath;
 
-    static inline std::unique_ptr<SimpleIni> create(const std::string& path) { return create(path, ""); }
-    static std::unique_ptr<SimpleIni>        create(const std::string& path, const std::string& defContent);
+    static inline std::shared_ptr<SimpleIni> create(const std::string& path) { return create(path, ""); }
+    static std::shared_ptr<SimpleIni>        create(const std::string& path, const std::string& defContent);
 
     bool        setInt(const std::string& sec, const std::string& key, int value);
     bool        setFloat(const std::string& sec, const std::string& key, float value);