From 92860503ce3aeee98c5793d5d8915534e9fd4f1d Mon Sep 17 00:00:00 2001
From: Giuseppe D'Angelo <giuseppe.dangelo@kdab.com>
Date: Fri, 28 Mar 2025 12:02:42 +0100
Subject: [PATCH] QHttpHeaders: remove an unnecessary std::move

The local is always moved in this situation. (I'm not sure why
this function returns a local but other functions use make_optional.)

Amends ed341fcdd18fb4bb58ea4fa9df0be6350a81578c.

Change-Id: I6719691ad13c65af083f1749be475dc8e2da4b07
Reviewed-by: Marc Mutz <marc.mutz@qt.io>
---
 src/network/access/qhttpheaders.cpp | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/src/network/access/qhttpheaders.cpp b/src/network/access/qhttpheaders.cpp
index 8f662c59a68..a5c2fe1411f 100644
--- a/src/network/access/qhttpheaders.cpp
+++ b/src/network/access/qhttpheaders.cpp
@@ -1532,7 +1532,7 @@ std::optional<QDateTime> QHttpHeaders::dateTimeValue(QAnyStringView name) const
         return std::nullopt;
     QDateTime dt = QNetworkHeadersPrivate::fromHttpDate(*v);
     if (dt.isValid())
-        return std::move(dt);
+        return dt;
     return std::nullopt;
 }