service/frd: don't use global system

address review comments + clang format
This commit is contained in:
Vitor Kiguchi 2021-07-07 05:09:33 -03:00
parent 6d99b5d332
commit 3e71d68b94
2 changed files with 12 additions and 7 deletions

View file

@ -16,6 +16,8 @@
#include "core/hle/service/frd/frd_a.h" #include "core/hle/service/frd/frd_a.h"
#include "core/hle/service/frd/frd_u.h" #include "core/hle/service/frd/frd_u.h"
SERVICE_CONSTRUCT_IMPL(Service::FRD::Module)
namespace Service::FRD { namespace Service::FRD {
Module::Interface::Interface(std::shared_ptr<Module> frd, const char* name, u32 max_session) 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 { struct ScreenName {
// 20 bytes according to 3dbrew // 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!"); ASSERT_MSG(cfg, "CFG Module missing!");
auto username = cfg->GetUsername(); auto username = cfg->GetUsername();
ASSERT_MSG(username.length() <= 10, "Username longer than expected!"); ASSERT_MSG(username.length() <= 10, "Username longer than expected!");
ScreenName screen_name{0}; ScreenName screen_name{};
std::memcpy(screen_name.name, username.data(), username.length() * sizeof(char16_t)); std::memcpy(screen_name.name.data(), username.data(), username.length() * sizeof(char16_t));
rb.Push(RESULT_SUCCESS); rb.Push(RESULT_SUCCESS);
rb.PushRaw(screen_name); 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); LOG_WARNING(Service_FRD, "(STUBBED) called, version: 0x{:08X}", version);
} }
Module::Module() = default; Module::Module(Core::System& system) : system(system){};
Module::~Module() = default; Module::~Module() = default;
void InstallInterfaces(Core::System& system) { void InstallInterfaces(Core::System& system) {
auto& service_manager = system.ServiceManager(); 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_U>(frd)->InstallAsService(service_manager);
std::make_shared<FRD_A>(frd)->InstallAsService(service_manager); std::make_shared<FRD_A>(frd)->InstallAsService(service_manager);
} }

View file

@ -50,7 +50,7 @@ struct Profile {
class Module final { class Module final {
public: public:
Module(); explicit Module(Core::System& system);
~Module(); ~Module();
class Interface : public ServiceFramework<Interface> { class Interface : public ServiceFramework<Interface> {
@ -153,6 +153,7 @@ public:
private: private:
FriendKey my_friend_key = {0, 0, 0ull}; FriendKey my_friend_key = {0, 0, 0ull};
MyPresence my_presence = {}; MyPresence my_presence = {};
Core::System& system;
template <class Archive> template <class Archive>
void serialize(Archive& ar, const unsigned int) { void serialize(Archive& ar, const unsigned int) {
@ -165,3 +166,5 @@ private:
void InstallInterfaces(Core::System& system); void InstallInterfaces(Core::System& system);
} // namespace Service::FRD } // namespace Service::FRD
SERVICE_CONSTRUCT(Service::FRD::Module)