Skip to content
Snippets Groups Projects
Commit da64da36 authored by Lioncash's avatar Lioncash
Browse files

services/sm: Amend error code constants

Courtesy of @ogniK5377.

This also moves them into the cpp file and limits the visibility to
where they're directly used. It also gets rid of unused or duplicate
error codes.
parent 8e7497d5
No related branches found
No related tags found
No related merge requests found
...@@ -15,6 +15,10 @@ ...@@ -15,6 +15,10 @@
namespace Service::SM { namespace Service::SM {
constexpr ResultCode ERR_ALREADY_REGISTERED(ErrorModule::SM, 4);
constexpr ResultCode ERR_INVALID_NAME(ErrorModule::SM, 6);
constexpr ResultCode ERR_SERVICE_NOT_REGISTERED(ErrorModule::SM, 7);
ServiceManager::ServiceManager() = default; ServiceManager::ServiceManager() = default;
ServiceManager::~ServiceManager() = default; ServiceManager::~ServiceManager() = default;
...@@ -24,10 +28,10 @@ void ServiceManager::InvokeControlRequest(Kernel::HLERequestContext& context) { ...@@ -24,10 +28,10 @@ void ServiceManager::InvokeControlRequest(Kernel::HLERequestContext& context) {
static ResultCode ValidateServiceName(const std::string& name) { static ResultCode ValidateServiceName(const std::string& name) {
if (name.size() <= 0 || name.size() > 8) { if (name.size() <= 0 || name.size() > 8) {
return ERR_INVALID_NAME_SIZE; return ERR_INVALID_NAME;
} }
if (name.find('\0') != std::string::npos) { if (name.find('\0') != std::string::npos) {
return ERR_NAME_CONTAINS_NUL; return ERR_INVALID_NAME;
} }
return RESULT_SUCCESS; return RESULT_SUCCESS;
} }
......
...@@ -36,12 +36,6 @@ private: ...@@ -36,12 +36,6 @@ private:
std::shared_ptr<ServiceManager> service_manager; std::shared_ptr<ServiceManager> service_manager;
}; };
constexpr ResultCode ERR_SERVICE_NOT_REGISTERED(-1);
constexpr ResultCode ERR_MAX_CONNECTIONS_REACHED(-1);
constexpr ResultCode ERR_INVALID_NAME_SIZE(-1);
constexpr ResultCode ERR_NAME_CONTAINS_NUL(-1);
constexpr ResultCode ERR_ALREADY_REGISTERED(-1);
class ServiceManager { class ServiceManager {
public: public:
static void InstallInterfaces(std::shared_ptr<ServiceManager> self); static void InstallInterfaces(std::shared_ptr<ServiceManager> self);
......
0% Loading or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment