diff options
author | Zach Hilman <zachhilman@gmail.com> | 2019-09-22 18:28:21 +0200 |
---|---|---|
committer | Zach Hilman <zachhilman@gmail.com> | 2019-09-22 18:34:55 +0200 |
commit | 6212df3beb38d0484b6ceedad18be99f107f7d96 (patch) | |
tree | fea1951fc95ba679834f2d690b19cc7b3de2e574 /src/core | |
parent | lm: Rename Initialize to Log and implement with manager/reporter (diff) | |
download | yuzu-6212df3beb38d0484b6ceedad18be99f107f7d96.tar yuzu-6212df3beb38d0484b6ceedad18be99f107f7d96.tar.gz yuzu-6212df3beb38d0484b6ceedad18be99f107f7d96.tar.bz2 yuzu-6212df3beb38d0484b6ceedad18be99f107f7d96.tar.lz yuzu-6212df3beb38d0484b6ceedad18be99f107f7d96.tar.xz yuzu-6212df3beb38d0484b6ceedad18be99f107f7d96.tar.zst yuzu-6212df3beb38d0484b6ceedad18be99f107f7d96.zip |
Diffstat (limited to 'src/core')
-rw-r--r-- | src/core/core.cpp | 2 | ||||
-rw-r--r-- | src/core/hle/service/lm/manager.cpp | 12 | ||||
-rw-r--r-- | src/core/hle/service/lm/manager.h | 4 | ||||
-rw-r--r-- | src/core/reporter.cpp | 5 | ||||
-rw-r--r-- | src/core/reporter.h | 3 |
5 files changed, 15 insertions, 11 deletions
diff --git a/src/core/core.cpp b/src/core/core.cpp index eb2c2e204..bf62085a8 100644 --- a/src/core/core.cpp +++ b/src/core/core.cpp @@ -250,6 +250,8 @@ struct System::Impl { telemetry_session->AddField(Telemetry::FieldType::Performance, "Mean_Frametime_MS", perf_stats->GetMeanFrametime()); + lm_manager.Flush(); + is_powered_on = false; exit_lock = false; diff --git a/src/core/hle/service/lm/manager.cpp b/src/core/hle/service/lm/manager.cpp index fd8edaf46..b67081b86 100644 --- a/src/core/hle/service/lm/manager.cpp +++ b/src/core/hle/service/lm/manager.cpp @@ -13,7 +13,7 @@ namespace Service::LM { std::ostream& operator<<(std::ostream& os, DestinationFlag dest) { std::vector<std::string> array; const auto check_single_flag = [dest, &array](DestinationFlag check, std::string name) { - if ((static_cast<u32>(check) & static_cast<u32>(dest)) > 0) { + if ((static_cast<u32>(check) & static_cast<u32>(dest)) != 0) { array.emplace_back(std::move(name)); } }; @@ -75,7 +75,7 @@ std::string FormatField(Field type, const std::vector<u8>& data) { if (data.size() >= sizeof(u32)) { u32 line; std::memcpy(&line, data.data(), sizeof(u32)); - return fmt::format("{:08X}", line); + return fmt::format("{}", line); } return "[ERROR DECODING LINE NUMBER]"; case Field::Message: @@ -114,16 +114,20 @@ void Manager::Log(LogMessage message) { } } +void Manager::Flush() { + FinalizeLog(); +} + void Manager::InitializeLog() { current_log.clear(); - LOG_INFO(Service_LM, "Initialized new log session!"); + LOG_INFO(Service_LM, "Initialized new log session"); } void Manager::FinalizeLog() { reporter.SaveLogReport(static_cast<u32>(destination), std::move(current_log)); - LOG_INFO(Service_LM, "Finalized current log session!"); + LOG_INFO(Service_LM, "Finalized current log session"); } } // namespace Service::LM diff --git a/src/core/hle/service/lm/manager.h b/src/core/hle/service/lm/manager.h index af0a27257..544e636ba 100644 --- a/src/core/hle/service/lm/manager.h +++ b/src/core/hle/service/lm/manager.h @@ -81,7 +81,7 @@ std::string FormatField(Field type, const std::vector<u8>& data); class Manager { public: - Manager(Core::Reporter& reporter); + explicit Manager(Core::Reporter& reporter); ~Manager(); void SetEnabled(bool enabled); @@ -89,6 +89,8 @@ public: void Log(LogMessage message); + void Flush(); + private: void InitializeLog(); void FinalizeLog(); diff --git a/src/core/reporter.cpp b/src/core/reporter.cpp index a465d7421..6f4af77fd 100644 --- a/src/core/reporter.cpp +++ b/src/core/reporter.cpp @@ -8,7 +8,6 @@ #include <fmt/chrono.h> #include <fmt/format.h> #include <fmt/ostream.h> -#include <fmt/time.h> #include <json.hpp> #include "common/file_util.h" @@ -393,11 +392,11 @@ void Reporter::SaveLogReport(u32 destination, std::vector<Service::LM::LogMessag out["type"] = fmt::format("{}", kv.first); out["data"] = Service::LM::FormatField(kv.first, kv.second); - return std::move(out); + return out; }); out["fields"] = std::move(fields); - return std::move(out); + return out; }); out["log_messages"] = std::move(json_messages); diff --git a/src/core/reporter.h b/src/core/reporter.h index 6e51113fd..380941b1b 100644 --- a/src/core/reporter.h +++ b/src/core/reporter.h @@ -59,9 +59,6 @@ public: New, System, }; - // Used by prepo services - void SavePlayReport(u64 title_id, u64 process_id, std::vector<std::vector<u8>> data, - std::optional<u128> user_id = {}) const; void SavePlayReport(PlayReportType type, u64 title_id, std::vector<std::vector<u8>> data, std::optional<u64> process_id = {}, std::optional<u128> user_id = {}) const; |