diff options
author | paradust7 <102263465+paradust7@users.noreply.github.com> | 2022-05-04 11:55:01 -0700 |
---|---|---|
committer | GitHub <noreply@github.com> | 2022-05-04 20:55:01 +0200 |
commit | 0704ca055059088bdd53e15be672e6b5663b8f50 (patch) | |
tree | 26d12dc712e19fcc38e3a70ef598efc665ab0d7f /src/network/connectionthreads.cpp | |
parent | ae7664597ed15f9ac779a9bac0595ab4125457c4 (diff) | |
download | hax-minetest-server-0704ca055059088bdd53e15be672e6b5663b8f50.tar.gz hax-minetest-server-0704ca055059088bdd53e15be672e6b5663b8f50.zip |
Make logging cost free when there is no output target (#12247)
The logging streams now do almost no work when there is no output target for them.
For example, if LL_VERBOSE has no output targets, then `verbosestream << x` will return a StreamProxy with a null target. Any further `<<` operations applied to it will do nothing.
Diffstat (limited to 'src/network/connectionthreads.cpp')
-rw-r--r-- | src/network/connectionthreads.cpp | 10 |
1 files changed, 3 insertions, 7 deletions
diff --git a/src/network/connectionthreads.cpp b/src/network/connectionthreads.cpp index dca065ae1..90936b43d 100644 --- a/src/network/connectionthreads.cpp +++ b/src/network/connectionthreads.cpp @@ -32,22 +32,18 @@ namespace con /* defines used for debugging and profiling */ /******************************************************************************/ #ifdef NDEBUG -#define LOG(a) a #define PROFILE(a) #undef DEBUG_CONNECTION_KBPS #else /* this mutex is used to achieve log message consistency */ -std::mutex log_conthread_mutex; -#define LOG(a) \ - { \ - MutexAutoLock loglock(log_conthread_mutex); \ - a; \ - } #define PROFILE(a) a //#define DEBUG_CONNECTION_KBPS #undef DEBUG_CONNECTION_KBPS #endif +// TODO: Clean this up. +#define LOG(a) a + #define WINDOW_SIZE 5 static session_t readPeerId(const u8 *packetdata) |