Merge pull request #3 from jmont-dev/memory-leak

jmont-dev · web-flow · commit ed608883d63c · 2024-08-01T02:06:42.000-04:00
Delete httplib when Ollama class is destroyed.
diff --git a/include/ollama.hpp b/include/ollama.hpp
@@ -374,7 +374,7 @@ class Ollama
         }
 
         Ollama(): Ollama("http://localhost:11434") {}
-        ~Ollama() {}
+        ~Ollama() { delete this->cli; }
 
     // Generate a non-streaming reply as a string.
     ollama::response generate(const std::string& model,const std::string& prompt, const json& options=nullptr, const std::vector<std::string>& images=std::vector<std::string>())

Original file line number	Diff line number	Diff line change
`@@ -374,7 +374,7 @@ class Ollama`
`374`	`374`	`}`
`375`	`375`
`376`	`376`	`Ollama(): Ollama("http://localhost:11434") {}`
`377`		`- ~Ollama() {}`
	`377`	`+ ~Ollama() { delete this->cli; }`
`378`	`378`
`379`	`379`	`// Generate a non-streaming reply as a string.`
`380`	`380`	`ollama::response generate(const std::string& model,const std::string& prompt, const json& options=nullptr, const std::vector<std::string>& images=std::vector<std::string>())`