ci(build): use Ninja as CMake generator explicit (#158)

Author: JBenda

.github/workflows/build.yml

@@ -72,9 +72,13 @@ class callback final : public callback_base
 			    static_cast<int>(type), static_cast<int>(new_val.type)
 			);
 			if constexpr (traits::arity == 2) {
-				// inkAssert(!old_val.has_value() || old_val.value().type == type,
-				// 	"Missmatch type for variable observers old value: expected optional<%i> got
-				// optional<%i>", static_cast<int>(type), static_cast<int>(old_val.value().type));
+				if (old_val.has_value() && old_val.value().type != type) {
+					inkFail(
+					    "Missmatch type for variable observers old value: expected optional<%i> got "
+					    "optional<%i>",
+					    static_cast<int>(type), static_cast<int>(old_val.value().type)
+					);
+				}
 			}
 		};
 		if constexpr (traits::arity > 0) {

inkcpp/include/functional.h

@@ -28,7 +28,7 @@ class globals_interface
 	 * @return nullopt if variable won't exist or type won't match
 	 */
 	template<typename T>
-	optional<T> get(const char* name) const
+	optional<T> get(const char* /*name*/) const
 	{
 		static_assert(internal::always_false<T>::value, "Requested Type is not supported");
 	}
@@ -41,7 +41,7 @@ class globals_interface
 	 * @retval true on success
 	 */
 	template<typename T>
-	bool set(const char* name, const T& val)
+	bool set(const char* /*name*/, const T& /*val*/)
 	{
 		static_assert(internal::always_false<T>::value, "Requested Type is not supported");
 		return false;

inkcpp/include/globals.h

@@ -129,6 +129,10 @@ struct value {
 
 	/// @}
 
+#ifdef __GNUCC__
+#	pragma GCC diagnostic push
+#	pragma GCC diagnostic ignored "-Wtautological-compare"
+#endif
 	/** Get value to corresponding type
 	 * @tparam Ty #Type label of type to get
 	 * @attention behavior if undefined if Ty != value.type
@@ -138,6 +142,9 @@ struct value {
 	{
 		static_assert(Ty != Ty, "No value getter for the selected type");
 	}
+#ifdef __GNUCC__
+#	pragma GCC diagnostic pop
+#endif
 };
 
 /** access a @ref ink::runtime::value::Type::Bool value */

inkcpp/include/types.h

@@ -60,7 +60,7 @@ class operation
 	static constexpr bool enabled = false;
 
 	template<typename T>
-	operation(const T& t)
+	operation(const T& /*t*/)
 	{
 	}
 
@@ -69,7 +69,10 @@ class operation
 	 * @param stack were the result(s) get pushed
 	 * @param vs array of values, first one = first argument etc
 	 */
-	void operator()(basic_eval_stack& stack, value* vs) { inkFail("operation not implemented!"); }
+	void operator()(basic_eval_stack& /*stack*/, value* /*vs*/)
+	{
+		inkFail("operation not implemented!");
+	}
 };
 } // namespace ink::runtime::internal
 

inkcpp/operations.h

@@ -120,7 +120,7 @@ void basic_stream::append(const value* in, unsigned int length)
 }
 
 template<typename T>
-inline void write_char(T& output, char c)
+inline void write_char(T& /*output*/, char /*c*/)
 {
 	static_assert(always_false<T>::value, "Invalid output type");
 }

inkcpp/output.cpp

@@ -31,15 +31,13 @@ namespace casting
 		const char* get() const { return _str; }
 
 	private:
-		const value& _val;
-		const char*  _str;
-		char         _data[512]; // TODO define central
+		const char* _str;
+		char        _data[512]; // TODO define central
 	};
 
 	// constructor for string_cast class
 	string_cast::string_cast(const value& val)
-	    : _val{val}
-	    , _str{nullptr}
+	    : _str{nullptr}
 	{
 		if (val.type() == value_type::string) {
 			// reference string if value is already a string

inkcpp/string_operations.cpp

@@ -17,7 +17,7 @@ namespace ink::runtime::internal
 class string_table final : public snapshot_interface
 {
 public:
-	virtual ~string_table();
+	~string_table();
 
 	// Create a dynamic string of a particular length
 	char* create(size_t length);

inkcpp/string_table.h

@@ -134,19 +134,27 @@ class value : public snapshot_interface
 	bool                set(const ink::runtime::value& val);
 	ink::runtime::value to_interface_value(list_table&) const;
 
+#ifdef __GNUCC__
+#	pragma GCC diagnostic push
+#	pragma GCC diagnostic ignored "-Wtautological-compare"
+#endif
 	/// get value of the type (if possible)
 	template<value_type ty>
 	typename ret<ty>::type get() const
 	{
 		static_assert(ty != ty, "No getter for this type defined!");
 	}
 
+#ifdef __GNUCC__
+#	pragma GCC diagnostic pop
+#endif
+
 	/// check if value evaluates to true
 	bool truthy(const list_table& lists) const;
 
 	/// set value of type (if possible)
 	template<value_type ty, typename... Args>
-	constexpr value& set(Args... args)
+	constexpr value& set(Args...)
 	{
 		static_assert(sizeof...(Args) != sizeof...(Args), "No setter for this type defined!");
 		return *this;
@@ -192,30 +200,37 @@ class value : public snapshot_interface
 	}
 
 	/// actual storage
+	struct value_jump {
+		uint32_t jump;
+		uint32_t thread_id;
+	};
+
+	struct value_frame {
+		uint32_t addr;
+		bool     eval; // was eval mode active in frame above
+	};
+
+	struct value_pointer {
+		hash_t name;
+		int    ci;
+	};
+
 	union {
+
 		bool        bool_value;
 		int32_t     int32_value;
 		string_type string_value;
 		uint32_t    uint32_value;
 		float       float_value;
 
-		struct {
-			uint32_t jump;
-			uint32_t thread_id;
-		} jump;
+		value_jump jump;
 
 		list_table::list list_value;
 		list_flag        list_flag_value;
 
-		struct {
-			uint32_t addr;
-			bool     eval; // was eval mode active in frame above
-		} frame_value;
+		value_frame frame_value;
 
-		struct {
-			hash_t name;
-			int    ci;
-		} pointer;
+		value_pointer pointer;
 	};
 
 	static constexpr size_t max_value_size = sizeof_largest_type<

inkcpp/value.h

@@ -1,7 +1,5 @@
 #include <string.h>
 #include <math.h>
-#include <stdlib.h>
-#include <stdio.h>
 
 #include <inkcpp.h>
 
@@ -12,7 +10,7 @@ HInkStory*   story  = NULL;
 HInkGlobals* store  = NULL;
 HInkRunner*  thread = NULL;
 
-void setup()
+void setup(void)
 {
 	if (! story) {
 		story = ink_story_from_file(INK_TEST_RESOURCE_DIR "GlobalStory.bin");
@@ -27,7 +25,7 @@ void setup()
 	thread = ink_story_new_runner(story, store);
 }
 
-int main()
+int main(void)
 {
 	//====== Just reading Globals =====
 	setup();
@@ -57,7 +55,7 @@ int main()
 	val.int32_v = 30;
 	assert(ink_globals_set(store, "age", val));
 
-	// set value of 'friendl_name_of_player'
+	// set value of 'friendly_name_of_player'
 	val.type     = ValueTypeString;
 	val.string_v = "Freddy";
 	assert(ink_globals_set(store, "friendly_name_of_player", val));