[LyX/master] Revert "use std::move() instead of copying when value is no longer used"

[Juergen Spitzmueller]

commit 30f170ea26023eef88918c9fe6da32605ee2a03d
Author: Juergen Spitzmueller <spitz at lyx.org>
Date:   Sat Sep 14 10:59:13 2024 +0200

    Revert "use std::move() instead of copying when value is no longer used"
    
    This reverts commit e685d31f5aadadd9c84c80a604818d49f5853dbf.
---
 src/BufferView.cpp          | 2 +-
 src/Encoding.cpp            | 2 +-
 src/insets/InsetNomencl.cpp | 2 +-
 3 files changed, 3 insertions(+), 3 deletions(-)

diff --git a/src/BufferView.cpp b/src/BufferView.cpp
index 63833ba8ca..a7348362cc 100644
--- a/src/BufferView.cpp
+++ b/src/BufferView.cpp
@@ -1707,7 +1707,7 @@ void BufferView::dispatch(FuncRequest const & cmd, DispatchResult & dr)
 			string label = dit.innerParagraph().getLabelForXRef();
 			if (!label.empty()) {
 				// if the paragraph has a label, we refer to this
-				string const arg = (type.empty()) ? move(label) : move(label) + " " + type;
+				string const arg = (type.empty()) ? label : label + " " + type;
 				lyx::dispatch(FuncRequest(LFUN_REFERENCE_INSERT, arg));
 				break;
 			} else {
diff --git a/src/Encoding.cpp b/src/Encoding.cpp
index 267298927f..cff17ee051 100644
--- a/src/Encoding.cpp
+++ b/src/Encoding.cpp
@@ -798,7 +798,7 @@ docstring Encodings::convertLaTeXCommands(docstring const & str, bool const lite
 		if (!cnvtd.empty()) {
 			// it did, so we'll take that bit and proceed with what's left
 			ret += cnvtd;
-			val = move(rem);
+			val = rem;
 			continue;
 		}
 		// it's a command of some sort
diff --git a/src/insets/InsetNomencl.cpp b/src/insets/InsetNomencl.cpp
index f853f14619..5db3c0ac87 100644
--- a/src/insets/InsetNomencl.cpp
+++ b/src/insets/InsetNomencl.cpp
@@ -440,7 +440,7 @@ docstring nomenclWidest(Buffer const & buffer)
 				msymb.size();
 			if (wx > w) {
 				w = wx;
-				symb = move(symbol);
+				symb = symbol;
 			}
 		}
 	}