Don't include fmt.h in text_types.h

Author: edo9300

gframe/Android/COSAndroidOperator.cpp

@@ -20,6 +20,7 @@ static inline int android_get_device_api_level() {
 #endif
 #include "COSAndroidOperator.h"
 #include "porting_android.h"
+#include "../fmt.h"
 
 namespace irr {
 

gframe/CGUISkinSystem/CGUISkinSystem.cpp

@@ -8,6 +8,7 @@
 #include <string>
 #include <set>
 #include "../utils.h"
+#include "../fmt.h"
 
 using namespace irr;
 
@@ -119,7 +120,7 @@ gui::CImageGUISkin* CGUISkinSystem::loadSkinFromFile(const fschar_t *skinname) {
 	// is to make the registry write out the root node and see what comes out.
 	//registry->writeFile("Skin",".");
 
-	// To switch on the fly, we have to set the skin to the fallback skin first	
+	// To switch on the fly, we have to set the skin to the fallback skin first
 	tmp = registry->getValueAsCStr(L"skin", L"Skin/Properties/Fallback");
 	// Always default to metalic
 	fallbackType = gui::EGST_WINDOWS_METALLIC;
@@ -156,7 +157,7 @@ gui::CImageGUISkin* CGUISkinSystem::loadSkinFromFile(const fschar_t *skinname) {
 	skinConfig.CheckBoxColor = registry->getValueAsColor(L"Skin/Global/CheckBoxColor");
 
 	// If there was no progress bar colors set, set them in the config to the defaults
-	// otherwise their 0,0,0,0. This is neccicary for the old klagui.	
+	// otherwise their 0,0,0,0. This is neccicary for the old klagui.
 
 	if(skinConfig.ProgressBarFilled.Color == 0)
 		skinConfig.ProgressBarFilled.Color = video::SColor(255, 255, 0, 0);
@@ -380,10 +381,10 @@ bool CGUISkinSystem::applySkin(const fschar_t *skinname) {
 	}
 	device->getGUIEnvironment()->setSkin(skin);
 	loaded_skin = skin;
-	// If we're going to switch skin we need to find all the progress bars and overwrite their colors	
+	// If we're going to switch skin we need to find all the progress bars and overwrite their colors
 	skin->drop();
 	registry = nullptr;
 	//fs->changeWorkingDirectoryTo(oldpath);
 
 	return true;
-}
+}

gframe/CGUITTFont/CGUITTFont.cpp

@@ -40,6 +40,7 @@
 #include <IMeshSceneNode.h>
 #include <unordered_set>
 #include <climits>
+#include "../fmt.h"
 #ifdef YGOPRO_USE_BUNDLED_FONT
 extern const char* bundled_font_name;
 extern const size_t bundled_font_len;

gframe/address.cpp

@@ -2,6 +2,7 @@
 #include "address.h"
 #include "bufferio.h"
 #include "config.h"
+#include "fmt.h"
 #if EDOPRO_ANDROID
 #include <netinet/in.h>
 #endif
@@ -42,19 +43,17 @@ void Address::toIn6Addr(in6_addr& sin6_addr) const {
 	memcpy(sin6_addr.s6_addr, buffer, sizeof(in6_addr::s6_addr));
 }
 
-template<>
-std::basic_string<char> Address::format() const {
-	if(family == UNK)
+std::string format_address(const Address& address) {
+	if(address.family == address.UNK)
 		return "";
 	char ret[50]{};
-	if(evutil_inet_ntop(family == INET ? AF_INET : AF_INET6, buffer, ret, sizeof(ret)) == nullptr)
+	if(evutil_inet_ntop(address.family == address.INET ? AF_INET : AF_INET6, address.buffer, ret, sizeof(ret)) == nullptr)
 		return "";
 	return ret;
 }
 
-template<>
-std::basic_string<wchar_t> Address::format() const {
-	return BufferIO::DecodeUTF8(format<char>());
+std::wstring wformat_address(const Address& address) {
+	return BufferIO::DecodeUTF8(format_address(address));
 }
 
 bool Host::operator==(const Host& other) const {

gframe/address.h

@@ -3,7 +3,6 @@
 
 #include <cstdint>
 #include <string>
-#include "fmt.h"
 #include "bufferio.h"
 #include "text_types.h"
 
@@ -16,6 +15,7 @@ namespace epro {
 
 struct Address {
 	friend struct Host;
+	friend std::string format_address(const Address& address);
 private:
 	uint8_t buffer[32]{}; //buffer big enough to store an ipv6
 public:
@@ -30,8 +30,6 @@ struct Address {
 	AF family;
 	void setIP4(const uint32_t* ip);
 	void setIP6(const void* ip);
-	template<typename T>
-	std::basic_string<T> format() const;
 	void toInAddr(in_addr& sin_addr) const;
 	void toIn6Addr(in6_addr& sin6_addr) const;
 };
@@ -46,22 +44,9 @@ struct Host {
 	static Host resolve(epro::stringview address, uint16_t port);
 };
 
-template<>
-std::basic_string<char> Address::format() const;
-template<>
-std::basic_string<wchar_t> Address::format() const;
-}
-
-template<typename T>
-struct fmt::formatter<epro::Address, T> {
-	template<typename ParseContext>
-	constexpr auto parse(ParseContext& ctx) const { return ctx.begin(); }
+std::string format_address(const Address&);
+std::wstring wformat_address(const Address&);
 
-	template<typename FormatContext>
-	auto format(const epro::Address& address, FormatContext& ctx) const {
-		static constexpr auto format_str = CHAR_T_STRINGVIEW(T, "{}");
-		return format_to(ctx.out(), format_str, address.format<T>());
-	}
-};
+}
 
 #endif //ADDRESS_H

gframe/bufferio.h

@@ -1,6 +1,7 @@
 #ifndef BUFFERIO_H
 #define BUFFERIO_H
 
+#include <algorithm>
 #include <string>
 #include <vector>
 #include <cstring>

gframe/client_card.cpp

@@ -2,6 +2,7 @@
 #include "data_manager.h"
 #include "common.h"
 #include "client_card.h"
+#include "fmt.h"
 
 namespace ygo {
 

gframe/client_field.cpp

@@ -21,6 +21,7 @@
 #include "CGUIImageButton/CGUIImageButton.h"
 #include "CGUITTFont/CGUITTFont.h"
 #include "custom_skin_enum.h"
+#include "fmt.h"
 
 namespace ygo {
 

gframe/client_updater.cpp

@@ -20,6 +20,7 @@
 #include "utils.h"
 #include "porting.h"
 #include "game_config.h"
+#include "fmt.h"
 
 #define LOCKFILE EPRO_TEXT("./.edopro_lock")
 #define UPDATES_FOLDER EPRO_TEXT("./updates/{}")

gframe/data_handler.cpp

@@ -6,6 +6,7 @@
 #include "utils_gui.h"
 #include "deck_manager.h"
 #include "logging.h"
+#include "fmt.h"
 #include "utils.h"
 #include "windbot.h"
 #include "windbot_panel.h"