Changed KeyValues.load_from_file() to a static method

Author: Ayuto

src/core/modules/keyvalues/keyvalues.h

@@ -106,8 +106,15 @@ class KeyValuesExt
 	static void SetBool(KeyValues* pKeyValues, const char * szName, bool bValue)
 	{ pKeyValues->SetInt(szName, bValue); }
 
-	static bool LoadFromFile(KeyValues* pKeyValues, const char * szFile)
-	{ return pKeyValues->LoadFromFile(filesystem, szFile); }
+	static boost::shared_ptr<KeyValues> LoadFromFile(const char * szFile)
+	{ 
+		KeyValues* pKeyValues = new KeyValues("");
+		if (!pKeyValues->LoadFromFile(filesystem, szFile)) {
+			pKeyValues->deleteThis();
+			return NULL;
+		}
+		return boost::shared_ptr<KeyValues>(pKeyValues, &__del__);
+	}
 
 	static bool SaveToFile(KeyValues* pKeyValues, const char * szFile)
 	{ return pKeyValues->SaveToFile(filesystem, szFile); }

src/core/modules/keyvalues/keyvalues_wrap.cpp

@@ -85,9 +85,9 @@ void export_keyvalues(scope _keyvalues)
 
 		.def("load_from_file",
 			&KeyValuesExt::LoadFromFile,
-			"Loads KeyValues data from a file into this KeyValues instance.",
+			"Load KeyValues data from a file and return a new KeyValues instance on success.",
 			args("file_name")
-		)
+		).staticmethod("load_from_file")
 
 		.def("save_to_file",
 			&KeyValuesExt::SaveToFile,