From f41f3781e1df7a9a64a05cc65e22488b7aa18d6d Mon Sep 17 00:00:00 2001
From: Sidharth Kshatriya <sid.kshatriya@gmail.com>
Date: Wed, 15 Mar 2023 12:34:12 +0530
Subject: [PATCH] Remove && from the template parameter given to declval

It seems redundant as declval already returns a rvalue reference
---
 src/optional.hh | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/src/optional.hh b/src/optional.hh
index e9386fa8..81dd15cf 100644
--- a/src/optional.hh
+++ b/src/optional.hh
@@ -95,7 +95,7 @@ public:
     template<typename U> using DecayOptional = typename DecayOptionalImpl<U>::Type;
 
     template<typename F>
-    auto map(F f) -> Optional<DecayOptional<decltype(f(std::declval<T&&>()))>>
+    auto map(F f) -> Optional<DecayOptional<decltype(f(std::declval<T>()))>>
     {
         if (not m_valid)
             return {};