Merge pull request #9134 from lioncash/init
audio_in/out_system: Pass Initialize members by value where applicable
This commit is contained in:
		@@ -56,7 +56,7 @@ Result System::IsConfigValid(const std::string_view device_name,
 | 
			
		||||
    return ResultSuccess;
 | 
			
		||||
}
 | 
			
		||||
 | 
			
		||||
Result System::Initialize(std::string& device_name, const AudioInParameter& in_params,
 | 
			
		||||
Result System::Initialize(std::string device_name, const AudioInParameter& in_params,
 | 
			
		||||
                          const u32 handle_, const u64 applet_resource_user_id_) {
 | 
			
		||||
    auto result{IsConfigValid(device_name, in_params)};
 | 
			
		||||
    if (result.IsError()) {
 | 
			
		||||
 
 | 
			
		||||
@@ -97,7 +97,7 @@ public:
 | 
			
		||||
     * @param applet_resource_user_id - Unused.
 | 
			
		||||
     * @return Result code.
 | 
			
		||||
     */
 | 
			
		||||
    Result Initialize(std::string& device_name, const AudioInParameter& in_params, u32 handle,
 | 
			
		||||
    Result Initialize(std::string device_name, const AudioInParameter& in_params, u32 handle,
 | 
			
		||||
                      u64 applet_resource_user_id);
 | 
			
		||||
 | 
			
		||||
    /**
 | 
			
		||||
 
 | 
			
		||||
@@ -49,8 +49,8 @@ Result System::IsConfigValid(std::string_view device_name,
 | 
			
		||||
    return Service::Audio::ERR_INVALID_CHANNEL_COUNT;
 | 
			
		||||
}
 | 
			
		||||
 | 
			
		||||
Result System::Initialize(std::string& device_name, const AudioOutParameter& in_params, u32 handle_,
 | 
			
		||||
                          u64& applet_resource_user_id_) {
 | 
			
		||||
Result System::Initialize(std::string device_name, const AudioOutParameter& in_params, u32 handle_,
 | 
			
		||||
                          u64 applet_resource_user_id_) {
 | 
			
		||||
    auto result = IsConfigValid(device_name, in_params);
 | 
			
		||||
    if (result.IsError()) {
 | 
			
		||||
        return result;
 | 
			
		||||
 
 | 
			
		||||
@@ -88,8 +88,8 @@ public:
 | 
			
		||||
     * @param applet_resource_user_id - Unused.
 | 
			
		||||
     * @return Result code.
 | 
			
		||||
     */
 | 
			
		||||
    Result Initialize(std::string& device_name, const AudioOutParameter& in_params, u32 handle,
 | 
			
		||||
                      u64& applet_resource_user_id);
 | 
			
		||||
    Result Initialize(std::string device_name, const AudioOutParameter& in_params, u32 handle,
 | 
			
		||||
                      u64 applet_resource_user_id);
 | 
			
		||||
 | 
			
		||||
    /**
 | 
			
		||||
     * Start this system.
 | 
			
		||||
 
 | 
			
		||||
@@ -17,7 +17,7 @@ using namespace AudioCore::AudioIn;
 | 
			
		||||
class IAudioIn final : public ServiceFramework<IAudioIn> {
 | 
			
		||||
public:
 | 
			
		||||
    explicit IAudioIn(Core::System& system_, Manager& manager, size_t session_id,
 | 
			
		||||
                      std::string& device_name, const AudioInParameter& in_params, u32 handle,
 | 
			
		||||
                      const std::string& device_name, const AudioInParameter& in_params, u32 handle,
 | 
			
		||||
                      u64 applet_resource_user_id)
 | 
			
		||||
        : ServiceFramework{system_, "IAudioIn"},
 | 
			
		||||
          service_context{system_, "IAudioIn"}, event{service_context.CreateEvent("AudioInEvent")},
 | 
			
		||||
 
 | 
			
		||||
@@ -24,7 +24,7 @@ using namespace AudioCore::AudioOut;
 | 
			
		||||
class IAudioOut final : public ServiceFramework<IAudioOut> {
 | 
			
		||||
public:
 | 
			
		||||
    explicit IAudioOut(Core::System& system_, AudioCore::AudioOut::Manager& manager,
 | 
			
		||||
                       size_t session_id, std::string& device_name,
 | 
			
		||||
                       size_t session_id, const std::string& device_name,
 | 
			
		||||
                       const AudioOutParameter& in_params, u32 handle, u64 applet_resource_user_id)
 | 
			
		||||
        : ServiceFramework{system_, "IAudioOut", ServiceThreadType::CreateNew},
 | 
			
		||||
          service_context{system_, "IAudioOut"}, event{service_context.CreateEvent(
 | 
			
		||||
 
 | 
			
		||||
		Reference in New Issue
	
	Block a user