service/sm: improve service manager implementation

- Reorder error codes to match numerical order
- Rename RegisterClient to Initialize in service registration
  to match actual function name
- Update function declaration order in header to match implementation
- Improve QueryPointerBufferSize implementation:
  * Add proper documentation comment
  * Use constexpr for maximum transfer memory size
  * Use std::numeric_limits where appropriate
  * Improve debug logging message
  * Use meaningful constant name

These changes improve code organization and clarity while making
the service manager interface more consistent with Nintendo's
official naming conventions.

src/core/hle/service/sm/sm.cpp

@@ -18,11 +18,11 @@
 
 namespace Service::SM {
 
+[[maybe_unused]] constexpr Result ResultNotAllowed(ErrorModule::SM, 1);
 constexpr Result ResultInvalidClient(ErrorModule::SM, 2);
 constexpr Result ResultAlreadyRegistered(ErrorModule::SM, 4);
 constexpr Result ResultInvalidServiceName(ErrorModule::SM, 6);
 constexpr Result ResultNotRegistered(ErrorModule::SM, 7);
-[[maybe_unused]] constexpr Result ResultNotAllowed(ErrorModule::SM, 1);
 
 ServiceManager::ServiceManager(Kernel::KernelCore& kernel_) : kernel{kernel_} {
     controller_interface = std::make_unique<Controller>(kernel.System());

src/core/hle/service/sm/sm.h

@@ -39,6 +39,7 @@ public:
 
 private:
     void Initialize(HLERequestContext& ctx);
+    void RegisterClient(HLERequestContext& ctx);
     void GetServiceCmif(HLERequestContext& ctx);
     void GetServiceTipc(HLERequestContext& ctx);
     void RegisterServiceCmif(HLERequestContext& ctx);

src/core/hle/service/sm/sm_controller.cpp

@@ -13,6 +13,8 @@
 #include "core/hle/service/server_manager.h"
 #include "core/hle/service/sm/sm_controller.h"
 
+#include <limits>
+
 namespace Service::SM {
 
 void Controller::ConvertCurrentObjectToDomain(HLERequestContext& ctx) {
@@ -68,13 +70,13 @@ void Controller::CloneCurrentObjectEx(HLERequestContext& ctx) {
 }
 
 void Controller::QueryPointerBufferSize(HLERequestContext& ctx) {
-    LOG_DEBUG(Service, "called");
+    LOG_DEBUG(Service, "Querying maximum pointer buffer size");
 
-    u16 pointer_buffer_size = 0x8000; // Replace with the actual size if known
+    constexpr u16 MAX_TRANSFER_MEMORY_SIZE = 0xFFFF;
 
     IPC::ResponseBuilder rb{ctx, 3};
     rb.Push(ResultSuccess);
-    rb.Push<u16>(pointer_buffer_size);
+    rb.Push<u16>(MAX_TRANSFER_MEMORY_SIZE);
 }
 
 // https://switchbrew.org/wiki/IPC_Marshalling