use for descriptive names for when checks are made

This commit is contained in:
AdenKoperczak 2025-01-11 13:50:49 -05:00
parent 2f9908b54e
commit 87e43a528e
3 changed files with 6 additions and 6 deletions

View file

@ -94,7 +94,7 @@ PrivilegeChecker::PrivilegeChecker() : p(std::make_unique<PrivilegeChecker::Impl
PrivilegeChecker::~PrivilegeChecker() = default; PrivilegeChecker::~PrivilegeChecker() = default;
bool PrivilegeChecker::first_check() bool PrivilegeChecker::pre_settings_check()
{ {
if (p->hasAppData_ || !p->highPrivilege_) if (p->hasAppData_ || !p->highPrivilege_)
{ {
@ -107,7 +107,7 @@ bool PrivilegeChecker::first_check()
return result != QMessageBox::Yes; return result != QMessageBox::Yes;
} }
bool PrivilegeChecker::second_check() bool PrivilegeChecker::post_settings_check()
{ {
auto& highPrivilegeWarningEnabled = auto& highPrivilegeWarningEnabled =
settings::GeneralSettings::Instance().high_privilege_warning_enabled(); settings::GeneralSettings::Instance().high_privilege_warning_enabled();

View file

@ -14,8 +14,8 @@ public:
~PrivilegeChecker(); ~PrivilegeChecker();
// returning true means check failed. // returning true means check failed.
bool first_check(); bool pre_settings_check();
bool second_check(); bool post_settings_check();
private: private:
class Impl; class Impl;

View file

@ -82,7 +82,7 @@ int main(int argc, char* argv[])
// Test to see if scwx was run with high privilege // Test to see if scwx was run with high privilege
scwx::qt::main::PrivilegeChecker privilegeChecker; scwx::qt::main::PrivilegeChecker privilegeChecker;
if (privilegeChecker.first_check()) if (privilegeChecker.pre_settings_check())
{ {
return 0; return 0;
} }
@ -127,7 +127,7 @@ int main(int argc, char* argv[])
scwx::qt::main::CheckProcessModules(); scwx::qt::main::CheckProcessModules();
int result = 0; int result = 0;
if (privilegeChecker.second_check()) if (privilegeChecker.post_settings_check())
{ {
result = 1; result = 1;
} }