summaryrefslogtreecommitdiffstats
path: root/src
diff options
context:
space:
mode:
authorbunnei <bunneidev@gmail.com>2018-07-15 21:58:52 +0200
committerGitHub <noreply@github.com>2018-07-15 21:58:52 +0200
commit04b9cde4f5a59e55ae7daac20f79121f8c7e0386 (patch)
tree64528e1b2ced65cf0f9a1a22ffaedabb069fa13d /src
parentMerge pull request #666 from bunnei/g8r8 (diff)
parentLogging: Dump all logs in the queue on close in debug mode (diff)
downloadyuzu-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.cpp11
-rw-r--r--src/common/logging/filter.cpp7
-rw-r--r--src/common/logging/filter.h3
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;
};