Upgrade djson to v0.1

- Emit all compile warnings as errors except on Debug.

Author: karnkaul

CMakeLists.txt

@@ -35,10 +35,11 @@ add_executable(${PROJECT_NAME} ${SOURCES})
 target_include_directories(${PROJECT_NAME} PRIVATE libs/include)
 
 if(CMAKE_CXX_COMPILER_ID STREQUAL Clang OR CMAKE_CXX_COMPILER_ID STREQUAL GNU)
-	target_compile_options(${PROJECT_NAME} PRIVATE -Wall -Wextra)
+	target_compile_options(${PROJECT_NAME} PRIVATE -Wall -Wextra $<$<NOT:$<CONFIG:Debug>>:-Werror>)
 endif()
 
-if(MSVC)
+if(CMAKE_CXX_COMPILER_ID STREQUAL "MSVC")
+	target_compile_options(${PROJECT_NAME} PRIVATE $<$<NOT:$<CONFIG:Debug>>:/WX>)
 	set_property(DIRECTORY ${CMAKE_SOURCE_DIR} PROPERTY VS_STARTUP_PROJECT ${PROJECT_NAME})
 endif()
 

libs/djson

@@ -4,7 +4,7 @@
 #include <vector>
 #include "utils.hpp"
 #include <str_format/str_format.hpp>
-#include <dumb_json/djson.hpp>
+#include <dumb_json/json.hpp>
 
 Move::Move(std::string name, MoveType type, int accuracy, int effect_chance, int power, std::string flavor_text)
 	: name(name), type(type), accuracy(accuracy), power(power), effect_chance(effect_chance), flavor_text(flavor_text)
@@ -47,24 +47,24 @@ Pokemon::Pokemon(int id, std::filesystem::path assets_dir) : assets_dir{assets_d
 {
 	this->sprite = read_asset("txt");
 	auto lines = read_asset("json");
-	this->json = std::accumulate(lines.begin(), lines.end(), std::string(""));
-	if (auto n = dj::node_t::make(this->json))
+	this->json_str = std::accumulate(lines.begin(), lines.end(), std::string(""));
+	dj::json_t json;
+	if (json.read(this->json_str) && json.is_object())
 	{
 		// pkmn data
-		dj::node_t& node = *n;
 		this->id = id;
-		this->name = node["name"].as<std::string>();
-		this->level = node["level"].as<int>();
-		this->hp = node["hp"].as<int>();
+		this->name = json["name"].as<std::string>();
+		this->level = json["level"].as<int>();
+		this->hp = json["hp"].as<int>();
 		this->max_hp = this->hp;
-		this->atk = node["atk"].as<int>();
-		this->def = node["def"].as<int>();
+		this->atk = json["atk"].as<int>();
+		this->def = json["def"].as<int>();
 		// move data
-		if (auto moves = node.find("moves"))
+		if (auto moves = json.find("moves"))
 		{
-			for (auto const& [num, m] : moves->as<dj::map_nodes_t>())
+			for (auto const& [num, m] : moves->as<dj::map_t>())
 			{
-				dj::node_t& move = *m;
+				dj::json_t& move = *m;
 
 				this->all_moves.push_back(
 					Move(

src/models.cpp

@@ -31,7 +31,7 @@ struct Pokemon
 	std::vector<std::string> read_asset(std::string ext);
 	void configure_move_set();
 
-	std::string json{};
+	std::string json_str{};
 	std::vector<Move> all_moves{};
 
   public:

src/models.hpp

@@ -9,7 +9,7 @@
 #include <thread>
 #include "utils.hpp"
 #include <str_format/str_format.hpp>
-#include <dumb_json/djson.hpp>
+#include <dumb_json/json.hpp>
 
 void clear_screen()
 {
@@ -74,14 +74,14 @@ Manifest check_manifest(const std::filesystem::path& path)
 
 	if (std::filesystem::exists(path))
 	{
-		auto file = read_file(path);
-		auto json = std::accumulate(file.begin(), file.end(), std::string(""));
-		if (auto n = dj::node_t::make(json))
+		auto const file = read_file(path);
+		auto const str = std::accumulate(file.begin(), file.end(), std::string(""));
+		dj::json_t json;
+		if (json.read(str) && json.is_object())
 		{
-			dj::node_t& node = *n;
-			manifest.game_ready = node["game_ready"].as<bool>();
-			manifest.duplicates = node["duplicates"].as<std::vector<int>>();
-			manifest.files = node["files"].as<std::vector<std::string>>();
+			manifest.game_ready = json["game_ready"].as<bool>();
+			manifest.duplicates = json["duplicates"].as<std::vector<int>>();
+			manifest.files = json["files"].as<std::vector<std::string>>();
 		}
 	}
 	else