diff options
author | bunnei <bunneidev@gmail.com> | 2018-07-15 21:58:52 +0200 |
---|---|---|
committer | GitHub <noreply@github.com> | 2018-07-15 21:58:52 +0200 |
commit | 04b9cde4f5a59e55ae7daac20f79121f8c7e0386 (patch) | |
tree | 64528e1b2ced65cf0f9a1a22ffaedabb069fa13d /src | |
parent | Merge pull request #666 from bunnei/g8r8 (diff) | |
parent | Logging: Dump all logs in the queue on close in debug mode (diff) | |
download | yuzu-04b9cde4f5a59e55ae7daac20f79121f8c7e0386.tar yuzu-04b9cde4f5a59e55ae7daac20f79121f8c7e0386.tar.gz yuzu-04b9cde4f5a59e55ae7daac20f79121f8c7e0386.tar.bz2 yuzu-04b9cde4f5a59e55ae7daac20f79121f8c7e0386.tar.lz yuzu-04b9cde4f5a59e55ae7daac20f79121f8c7e0386.tar.xz yuzu-04b9cde4f5a59e55ae7daac20f79121f8c7e0386.tar.zst yuzu-04b9cde4f5a59e55ae7daac20f79121f8c7e0386.zip |
Diffstat (limited to 'src')
-rw-r--r-- | src/common/logging/backend.cpp | 11 | ||||
-rw-r--r-- | src/common/logging/filter.cpp | 7 | ||||
-rw-r--r-- | src/common/logging/filter.h | 3 |
3 files changed, 17 insertions, 4 deletions
diff --git a/src/common/logging/backend.cpp b/src/common/logging/backend.cpp index 242914c6a..ed1e93cc2 100644 --- a/src/common/logging/backend.cpp +++ b/src/common/logging/backend.cpp @@ -5,6 +5,7 @@ #include <algorithm> #include <array> #include <chrono> +#include <climits> #include <condition_variable> #include <memory> #include <thread> @@ -83,8 +84,10 @@ private: } }; while (true) { - std::unique_lock<std::mutex> lock(message_mutex); - message_cv.wait(lock, [&] { return !running || message_queue.Pop(entry); }); + { + std::unique_lock<std::mutex> lock(message_mutex); + message_cv.wait(lock, [&] { return !running || message_queue.Pop(entry); }); + } if (!running) { break; } @@ -92,7 +95,7 @@ private: } // Drain the logging queue. Only writes out up to MAX_LOGS_TO_WRITE to prevent a case // where a system is repeatedly spamming logs even on close. - constexpr int MAX_LOGS_TO_WRITE = 100; + const int MAX_LOGS_TO_WRITE = filter.IsDebug() ? INT_MAX : 100; int logs_written = 0; while (logs_written++ < MAX_LOGS_TO_WRITE && message_queue.Pop(entry)) { write_logs(entry); @@ -282,4 +285,4 @@ void FmtLogMessageImpl(Class log_class, Level log_level, const char* filename, Impl::Instance().PushEntry(std::move(entry)); } -} // namespace Log
\ No newline at end of file +} // namespace Log diff --git a/src/common/logging/filter.cpp b/src/common/logging/filter.cpp index 4e783a577..6ed087beb 100644 --- a/src/common/logging/filter.cpp +++ b/src/common/logging/filter.cpp @@ -94,4 +94,11 @@ bool Filter::ParseFilterRule(const std::string::const_iterator begin, bool Filter::CheckMessage(Class log_class, Level level) const { return static_cast<u8>(level) >= static_cast<u8>(class_levels[static_cast<size_t>(log_class)]); } + +bool Filter::IsDebug() const { + return std::any_of(class_levels.begin(), class_levels.end(), [](const Level& l) { + return static_cast<u8>(l) <= static_cast<u8>(Level::Debug); + }); +} + } // namespace Log diff --git a/src/common/logging/filter.h b/src/common/logging/filter.h index ccca289bd..2a4f7c845 100644 --- a/src/common/logging/filter.h +++ b/src/common/logging/filter.h @@ -47,6 +47,9 @@ public: /// Matches class/level combination against the filter, returning true if it passed. bool CheckMessage(Class log_class, Level level) const; + /// Returns true if any logging classes are set to debug + bool IsDebug() const; + private: std::array<Level, (size_t)Class::Count> class_levels; }; |