Clean up warnings and unused parameters.

Author: jmont-dev

Makefile

@@ -1,19 +1,22 @@
 # Check if a default C++ compiler exists, otherwise use g++
 CXX ?= g++
+CXXFLAGS = -Wall -Wextra -Wpedantic
+
+CREATE_BUILD_DIR = mkdir -p build; cp -n llama.jpg build;
 
 all: examples test-cpp11 test-cpp14 test-cpp20
 examples: examples/main.cpp
-	mkdir -p build
-	$(CXX) examples/main.cpp -Iinclude -o build/examples -std=c++11 -pthread -latomic
+	$(CREATE_BUILD_DIR)
+	$(CXX) $(CXXFLAGS) examples/main.cpp -Iinclude -o build/examples -std=c++11 -pthread -latomic
 test: test-cpp11
 test-cpp11: test/test.cpp
-	mkdir -p build
-	$(CXX) test/test.cpp -Iinclude -Itest -o build/test -std=c++11 -pthread -latomic
+	$(CREATE_BUILD_DIR)
+	$(CXX) $(CXXFLAGS) test/test.cpp -Iinclude -Itest -o build/test -std=c++11 -pthread -latomic
 test-cpp14: test/test.cpp
-	mkdir -p build
-	$(CXX) test/test.cpp -Iinclude -Itest -o build/test-cpp14 -std=c++14 -pthread -latomic
+	$(CREATE_BUILD_DIR)
+	$(CXX) $(CXXFLAGS) test/test.cpp -Iinclude -Itest -o build/test-cpp14 -std=c++14 -pthread -latomic
 test-cpp20: test/test.cpp
-	mkdir -p build
-	$(CXX) test/test.cpp -Iinclude -Itest -o build/test-cpp20 -std=c++2a -pthread -latomic
+	$(CREATE_BUILD_DIR)
+	$(CXX) $(CXXFLAGS) test/test.cpp -Iinclude -Itest -o build/test-cpp20 -std=c++2a -pthread -latomic
 clean:
 	rm -rf build

include/ollama.hpp

@@ -101,7 +101,7 @@ namespace ollama
         public:
             image(const std::string base64_sequence, bool valid = true) 
             {
-                this->base64_sequence = base64_sequence;
+                this->base64_sequence = base64_sequence; this->valid = valid;
             }
             ~image(){};
 
@@ -254,7 +254,7 @@ namespace ollama
                 (*this)["stream"] = stream;
 
                 if (options!=nullptr) (*this)["options"] = options["options"];
-                //(*this)["format"] = format; // Commented out as providing the format causes issues with some models.
+                (void)format; //(*this)["format"] = format; // Commented out as providing the format causes issues with some models.
                 (*this)["keep_alive"] = keep_alive_duration;
                 type = message_type::chat;
 
@@ -329,7 +329,7 @@ namespace ollama
                 return simple_string;               
             }
 
-            const bool has_error() const
+            bool has_error() const
             {
                 if ( json_data.contains("error") ) return true;
                 return false;                
@@ -871,12 +871,13 @@ class Ollama
 
     private:
 
+/*
     bool send_request(const ollama::request& request, std::function<void(const ollama::response&)> on_receive_response=nullptr)
     {
 
         return true;
     }
-
+*/
 
     std::string server_url;
     httplib::Client *cli;
@@ -1039,7 +1040,7 @@ namespace ollama
         ollama.setWriteTimeout(seconds);
     }
 
-};
+}
 
 
 #endif

singleheader/ollama.hpp

@@ -34891,7 +34891,7 @@ namespace ollama
         public:
             image(const std::string base64_sequence, bool valid = true) 
             {
-                this->base64_sequence = base64_sequence;
+                this->base64_sequence = base64_sequence; this->valid = valid;
             }
             ~image(){};
 
@@ -35044,7 +35044,7 @@ namespace ollama
                 (*this)["stream"] = stream;
 
                 if (options!=nullptr) (*this)["options"] = options["options"];
-                //(*this)["format"] = format; // Commented out as providing the format causes issues with some models.
+                (void)format; //(*this)["format"] = format; // Commented out as providing the format causes issues with some models.
                 (*this)["keep_alive"] = keep_alive_duration;
                 type = message_type::chat;
 
@@ -35119,7 +35119,7 @@ namespace ollama
                 return simple_string;               
             }
 
-            const bool has_error() const
+            bool has_error() const
             {
                 if ( json_data.contains("error") ) return true;
                 return false;                
@@ -35661,12 +35661,13 @@ class Ollama
 
     private:
 
+/*
     bool send_request(const ollama::request& request, std::function<void(const ollama::response&)> on_receive_response=nullptr)
     {
 
         return true;
     }
-
+*/
 
     std::string server_url;
     httplib::Client *cli;
@@ -35829,7 +35830,7 @@ namespace ollama
         ollama.setWriteTimeout(seconds);
     }
 
-};
+}
 
 
 #endif

test/test.cpp

@@ -106,7 +106,7 @@ TEST_SUITE("Ollama Tests") {
         try { 
             ollama::generate("Non-existent-model", "Requesting this model will throw an error"); 
         } 
-        catch(ollama::exception e) { exception_handled = true; }
+        catch(ollama::exception& e) { exception_handled = true; }
 
         CHECK( exception_handled );
     }