From 133ebfe96385a0e94ab8b029a31ca0cd7a23c50a Mon Sep 17 00:00:00 2001
From: Dan Goodliffe <dan@randomdan.homeip.net>
Date: Sat, 1 May 2021 19:56:59 +0100
Subject: Pass new values more efficiently

---
 lib/persistence.h | 20 ++++++++++----------
 1 file changed, 10 insertions(+), 10 deletions(-)

(limited to 'lib/persistence.h')

diff --git a/lib/persistence.h b/lib/persistence.h
index e8e3415..5d5fb0f 100644
--- a/lib/persistence.h
+++ b/lib/persistence.h
@@ -28,10 +28,10 @@ namespace Persistence {
 		virtual ~Selection() = default;
 		DEFAULT_MOVE_COPY(Selection);
 
-		virtual void setValue(float &);
-		virtual void setValue(bool &);
-		virtual void setValue(const std::nullptr_t &);
-		virtual void setValue(std::string &);
+		virtual void setValue(float);
+		virtual void setValue(bool);
+		virtual void setValue(std::nullptr_t);
+		virtual void setValue(std::string &&);
 		virtual void beginArray(Stack &);
 		virtual void beginObject(Stack &);
 		virtual void endObject(Stack &);
@@ -67,9 +67,10 @@ namespace Persistence {
 
 	template<typename T> struct SelectionT : public SelectionV<T> {
 		using SelectionV<T>::SelectionV;
+		using P = std::conditional_t<std::is_scalar_v<T>, T, T &&>;
 
 		void
-		setValue(T & evalue) override
+		setValue(P evalue) override
 		{
 			std::swap(this->v, evalue);
 		}
@@ -190,7 +191,7 @@ namespace Persistence {
 				using SelectionV<Ptr>::SelectionV;
 
 				void
-				setValue(std::string & type) override
+				setValue(std::string && type) override
 				{
 					if constexpr (shared) {
 						auto no = Persistable::callSharedFactory(type);
@@ -214,7 +215,7 @@ namespace Persistence {
 				using SelectionV<Ptr>::SelectionV;
 
 				void
-				setValue(std::string & id) override
+				setValue(std::string && id) override
 				{
 					sharedObjects.emplace(id, this->v);
 				}
@@ -271,8 +272,7 @@ namespace Persistence {
 
 		using SelectionV<Ptr>::SelectionV;
 
-		void
-		setValue(const std::nullptr_t &) override
+		void setValue(std::nullptr_t) override
 		{
 			this->v.reset();
 		}
@@ -298,7 +298,7 @@ namespace Persistence {
 		using SelectionPtrBase<std::shared_ptr<T>, true>::SelectionPtrBase;
 
 		void
-		setValue(std::string & id) override
+		setValue(std::string && id) override
 		{
 			if (auto teo = std::dynamic_pointer_cast<T>(sharedObjects.at(id))) {
 				this->v = std::move(teo);
-- 
cgit v1.2.3