From 3e71d68b94964f913986e045ac998994c3cfe370 Mon Sep 17 00:00:00 2001
From: Vitor Kiguchi <vitor-kiguchi@hotmail.com>
Date: Wed, 7 Jul 2021 05:09:33 -0300
Subject: [PATCH] service/frd: don't use global system

address review comments + clang format
---
 src/core/hle/service/frd/frd.cpp | 14 ++++++++------
 src/core/hle/service/frd/frd.h   |  5 ++++-
 2 files changed, 12 insertions(+), 7 deletions(-)

diff --git a/src/core/hle/service/frd/frd.cpp b/src/core/hle/service/frd/frd.cpp
index 1e49a1e8bd..605fc9ab4d 100644
--- a/src/core/hle/service/frd/frd.cpp
+++ b/src/core/hle/service/frd/frd.cpp
@@ -16,6 +16,8 @@
 #include "core/hle/service/frd/frd_a.h"
 #include "core/hle/service/frd/frd_u.h"
 
+SERVICE_CONSTRUCT_IMPL(Service::FRD::Module)
+
 namespace Service::FRD {
 
 Module::Interface::Interface(std::shared_ptr<Module> frd, const char* name, u32 max_session)
@@ -96,15 +98,15 @@ void Module::Interface::GetMyScreenName(Kernel::HLERequestContext& ctx) {
 
     struct ScreenName {
         // 20 bytes according to 3dbrew
-        char16_t name[10];
+        std::array<char16_t, 10> name;
     };
 
-    auto cfg = Service::CFG::GetModule(Core::System::GetInstance());
+    auto cfg = Service::CFG::GetModule(frd->system);
     ASSERT_MSG(cfg, "CFG Module missing!");
     auto username = cfg->GetUsername();
     ASSERT_MSG(username.length() <= 10, "Username longer than expected!");
-    ScreenName screen_name{0};
-    std::memcpy(screen_name.name, username.data(), username.length() * sizeof(char16_t));
+    ScreenName screen_name{};
+    std::memcpy(screen_name.name.data(), username.data(), username.length() * sizeof(char16_t));
 
     rb.Push(RESULT_SUCCESS);
     rb.PushRaw(screen_name);
@@ -154,12 +156,12 @@ void Module::Interface::SetClientSdkVersion(Kernel::HLERequestContext& ctx) {
     LOG_WARNING(Service_FRD, "(STUBBED) called, version: 0x{:08X}", version);
 }
 
-Module::Module() = default;
+Module::Module(Core::System& system) : system(system){};
 Module::~Module() = default;
 
 void InstallInterfaces(Core::System& system) {
     auto& service_manager = system.ServiceManager();
-    auto frd = std::make_shared<Module>();
+    auto frd = std::make_shared<Module>(system);
     std::make_shared<FRD_U>(frd)->InstallAsService(service_manager);
     std::make_shared<FRD_A>(frd)->InstallAsService(service_manager);
 }
diff --git a/src/core/hle/service/frd/frd.h b/src/core/hle/service/frd/frd.h
index 1f34e61fa6..16902e45da 100644
--- a/src/core/hle/service/frd/frd.h
+++ b/src/core/hle/service/frd/frd.h
@@ -50,7 +50,7 @@ struct Profile {
 
 class Module final {
 public:
-    Module();
+    explicit Module(Core::System& system);
     ~Module();
 
     class Interface : public ServiceFramework<Interface> {
@@ -153,6 +153,7 @@ public:
 private:
     FriendKey my_friend_key = {0, 0, 0ull};
     MyPresence my_presence = {};
+    Core::System& system;
 
     template <class Archive>
     void serialize(Archive& ar, const unsigned int) {
@@ -165,3 +166,5 @@ private:
 void InstallInterfaces(Core::System& system);
 
 } // namespace Service::FRD
+
+SERVICE_CONSTRUCT(Service::FRD::Module)