C++17 features

Author: salix5

gframe/bufferio.h

@@ -143,7 +143,7 @@ class BufferIO {
 		while (*pw != 0) {
 			unsigned cur = 0;
 			int codepoint_size = 0;
-			if (sizeof(wchar_t) == 2) {
+			if constexpr (sizeof(wchar_t) == 2) {
 				if (IsHighSurrogate(pw[0])) {
 					if (pw[1] == 0)
 						break;
@@ -220,7 +220,7 @@ class BufferIO {
 			if (!IsUnicodeChar(cur))
 				continue;
 			if (cur >= 0x10000) {
-				if (sizeof(wchar_t) == 2)
+				if constexpr (sizeof(wchar_t) == 2)
 					codepoint_size = 2;
 				else
 					codepoint_size = 1;

gframe/duelclient.h

@@ -55,7 +55,7 @@ class DuelClient {
 	template<typename ST>
 	static void SendPacketToServer(unsigned char proto, const ST& st) {
 		auto p = duel_client_write;
-		if (sizeof(ST) > MAX_DATA_SIZE)
+		if constexpr (sizeof(ST) > MAX_DATA_SIZE)
 			return;
 		buffer_write<uint16_t>(p, (uint16_t)(1 + sizeof(ST)));
 		buffer_write<uint8_t>(p, proto);

gframe/irrUString.h

@@ -449,7 +449,7 @@ class ustring16 {
 	//! Returns the length of a ustring16 in full characters.
 	//! \return Length of a ustring16 in full characters.
 	u32 size() const {
-		if (sizeof(wchar_t) == 4) {
+		if constexpr (sizeof(wchar_t) == 4) {
 			return size_raw_;
 		} else {
 			if(size_ != 0xffffffff)
@@ -524,7 +524,7 @@ class ustring16 {
 	//! \return A reference to our current string.
 	void validate() {
 		// Validate all unicode characters.
-		if (sizeof(wchar_t) == 4) {
+		if constexpr (sizeof(wchar_t) == 4) {
 			size_ = size_raw_;
 			return;
 		}

gframe/netserver.h

@@ -43,7 +43,7 @@ class NetServer {
 	template<typename ST>
 	static void SendPacketToPlayer(DuelPlayer* dp, unsigned char proto, const ST& st) {
 		auto p = net_server_write;
-		if (sizeof(ST) > MAX_DATA_SIZE)
+		if constexpr (sizeof(ST) > MAX_DATA_SIZE)
 			return;
 		buffer_write<uint16_t>(p, (uint16_t)(1 + sizeof(ST)));
 		buffer_write<uint8_t>(p, proto);

gframe/network.h

@@ -10,7 +10,7 @@
 #include <event2/thread.h>
 #include <type_traits>
 
-#define check_trivially_copyable(T) static_assert(std::is_trivially_copyable<T>::value == true && std::is_standard_layout<T>::value == true, "not trivially copyable")
+#define check_trivially_copyable(T) static_assert(std::is_trivially_copyable_v<T> == true && std::is_standard_layout_v<T> == true, "not trivially copyable")
 
 namespace ygo {
 	constexpr int SIZE_NETWORK_BUFFER = 0x20000;