fix: avoid alias copy on return

Author: qubka

include/plugify/binding.hpp

@@ -25,14 +25,14 @@ namespace plugify {
 		[[nodiscard]] const std::string& GetMethod() const noexcept;
 		[[nodiscard]] bool IsBindSelf() const noexcept;
 		[[nodiscard]] const std::inplace_vector<std::optional<Alias>, Signature::kMaxFuncArgs>& GetParamAliases() const noexcept;
-		[[nodiscard]] std::optional<Alias> GetRetAlias() const noexcept;
+		[[nodiscard]] const std::optional<Alias>& GetRetAlias() const noexcept;
 
 		// Setters (pass by value and move)
 		void SetName(std::string name);
 		void SetMethod(std::string method);
 		void SetBindSelf(bool bindSelf);
 		void SetParamAliases(std::inplace_vector<std::optional<Alias>, Signature::kMaxFuncArgs> paramAliases);
-		void SetRetAlias(Alias retAlias);
+		void SetRetAlias(std::optional<Alias> retAlias);
 
 		[[nodiscard]] bool operator==(const Binding& other) const noexcept;
 		[[nodiscard]] auto operator<=>(const Binding& other) const noexcept;

src/core/binding.cpp

@@ -43,7 +43,7 @@ const std::inplace_vector<std::optional<Alias>, Signature::kMaxFuncArgs>& Bindin
 	return _impl->paramAliases ? *_impl->paramAliases : emptyAliases;
 }
 
-std::optional<Alias> Binding::GetRetAlias() const noexcept {
+const std::optional<Alias>& Binding::GetRetAlias() const noexcept {
 	return _impl->retAlias;
 }
 
@@ -63,7 +63,7 @@ void Binding::SetParamAliases(std::inplace_vector<std::optional<Alias>, Signatur
 	_impl->paramAliases = std::move(paramAliases);
 }
 
-void Binding::SetRetAlias(Alias retAlias) {
+void Binding::SetRetAlias(std::optional<Alias> retAlias) {
 	_impl->retAlias = std::move(retAlias);
 }