diff --git a/deps/discord-rpc/include/discord_rpc.h b/deps/discord-rpc/include/discord_rpc.h index ade0e2c372..3e1441e058 100644 --- a/deps/discord-rpc/include/discord_rpc.h +++ b/deps/discord-rpc/include/discord_rpc.h @@ -1,7 +1,7 @@ #pragma once #include -/* clang-format off */ +// clang-format off #if defined(DISCORD_DYNAMIC_LIB) # if defined(_WIN32) @@ -17,7 +17,7 @@ # define DISCORD_EXPORT #endif -/* clang-format on */ +// clang-format on #ifdef __cplusplus extern "C" { diff --git a/deps/discord-rpc/src/serialization.h b/deps/discord-rpc/src/serialization.h index d5e7e4a914..9c462dc283 100644 --- a/deps/discord-rpc/src/serialization.h +++ b/deps/discord-rpc/src/serialization.h @@ -5,12 +5,12 @@ #ifndef __MINGW32__ #pragma warning(push) -#pragma warning(disable : 4061) /* enum is not explicitly handled by a case label */ -#pragma warning(disable : 4365) /* signed/unsigned mismatch */ -#pragma warning(disable : 4464) /* relative include path contains */ -#pragma warning(disable : 4668) /* is not defined as a preprocessor macro */ -#pragma warning(disable : 6313) /* Incorrect operator*/ -#endif /* __MINGW32__ */ +#pragma warning(disable : 4061) // enum is not explicitly handled by a case label +#pragma warning(disable : 4365) // signed/unsigned mismatch +#pragma warning(disable : 4464) // relative include path contains +#pragma warning(disable : 4668) // is not defined as a preprocessor macro +#pragma warning(disable : 6313) // Incorrect operator +#endif // __MINGW32__ #include "rapidjson/document.h" #include "rapidjson/stringbuffer.h" @@ -18,26 +18,27 @@ #ifndef __MINGW32__ #pragma warning(pop) -#endif /* __MINGW32__ */ +#endif // __MINGW32__ -/* if only there was a standard library function for this */ +// if only there was a standard library function for this template inline size_t StringCopy(char (&dest)[Len], const char* src) { - size_t copied; - char *out; - if (!src || !Len) + if (!src || !Len) { return 0; - out = dest; - for (copied = 1; *src && copied < Len; ++copied) + } + size_t copied; + char* out = dest; + for (copied = 1; *src && copied < Len; ++copied) { *out++ = *src++; + } *out = 0; return copied - 1; } size_t JsonWriteHandshakeObj(char* dest, size_t maxLen, int version, const char* applicationId); -/* Commands */ +// Commands struct DiscordRichPresence; size_t JsonWriteRichPresenceObj(char* dest, size_t maxLen, @@ -50,9 +51,8 @@ size_t JsonWriteUnsubscribeCommand(char* dest, size_t maxLen, int nonce, const c size_t JsonWriteJoinReply(char* dest, size_t maxLen, const char* userId, int reply, int nonce); -/* I want to use as few allocations as I can get away with, and to do that with RapidJson, you need - * to supply some of your own allocators for stuff rather than use the defaults - */ +// I want to use as few allocations as I can get away with, and to do that with RapidJson, you need +// to supply some of your own allocators for stuff rather than use the defaults class LinearAllocator { public: @@ -60,7 +60,7 @@ public: char* end_; LinearAllocator() { - assert(0); /* needed for some default case in rapidjson, should not use */ + assert(0); // needed for some default case in rapidjson, should not use } LinearAllocator(char* buffer, size_t size) : buffer_(buffer) @@ -80,11 +80,12 @@ public: } void* Realloc(void* originalPtr, size_t originalSize, size_t newSize) { - if (newSize == 0) + if (newSize == 0) { return nullptr; - /* allocate how much you need in the first place */ + } + // allocate how much you need in the first place assert(!originalPtr && !originalSize); - /* unused parameter warning */ + // unused parameter warning (void)(originalPtr); (void)(originalSize); return Malloc(newSize); @@ -107,7 +108,7 @@ public: static const bool kNeedFree = false; }; -/* wonder why this isn't a thing already, maybe I missed it */ +// wonder why this isn't a thing already, maybe I missed it class DirectStringBuffer { public: using Ch = char; @@ -135,7 +136,7 @@ public: using MallocAllocator = rapidjson::CrtAllocator; using PoolAllocator = rapidjson::MemoryPoolAllocator; using UTF8 = rapidjson::UTF8; -/* Writer appears to need about 16 bytes per nested object level (with 64bit size_t) */ +// Writer appears to need about 16 bytes per nested object level (with 64bit size_t) using StackAllocator = FixedLinearAllocator<2048>; constexpr size_t WriterNestingLevels = 2048 / (2 * sizeof(size_t)); using JsonWriterBase = @@ -159,8 +160,8 @@ using JsonDocumentBase = rapidjson::GenericDocument