From 019a62e8bd98efcdc9a171d59b78eb98b6c7ecc2cf0c6d17aa5dd9d7ebe7eb72 Mon Sep 17 00:00:00 2001
From: Daniel Weatherill <plasteredparrot@gmail.com>
Date: Tue, 4 Jun 2019 02:55:31 +0000
Subject: [PATCH] - remove_pessimizing_moves.patch: remove NRVO pessimizing
 moves   (fixes a warning on gcc 9 to fix current Factory build)

OBS-URL: https://build.opensuse.org/package/show/devel:libraries:c_c++/rttr?expand=0&rev=8
---
 remove_pessimizing_moves.patch | 58 ++++++++++++++++++++++++++++++++++
 rttr.changes                   |  6 ++++
 rttr.spec                      |  3 ++
 3 files changed, 67 insertions(+)
 create mode 100644 remove_pessimizing_moves.patch

diff --git a/remove_pessimizing_moves.patch b/remove_pessimizing_moves.patch
new file mode 100644
index 0000000..8dface6
--- /dev/null
+++ b/remove_pessimizing_moves.patch
@@ -0,0 +1,58 @@
+Index: rttr-0.9.6/src/rttr/detail/registration/bind_impl.h
+===================================================================
+--- rttr-0.9.6.orig/src/rttr/detail/registration/bind_impl.h
++++ rttr-0.9.6/src/rttr/detail/registration/bind_impl.h
+@@ -289,7 +289,7 @@ class registration::bind<detail::ctor_fu
+                                                    std::move(get_metadata(std::forward<Args>(args)...)),
+                                                    std::move(get_default_args<type_list<Acc_Func>, function_type>(std::forward<Args>(args)...)),
+                                                    std::move(create_param_infos<type_list<F>, function_type>(std::forward<Args>(args)...)));
+-            return std::move(ctor);
++            return ctor;
+         }
+     public:
+         bind(const std::shared_ptr<detail::registration_executer>& reg_exec, F func)
+@@ -386,7 +386,7 @@ class registration::bind<detail::prop, C
+                                                              detail::map_access_level_to_enum<acc_level>::value,
+                                                              getter_policy, setter_policy,
+                                                              Metadata_Count>>(name, type::get<Class_Type>(), acc, std::move(metadata_list));
+-            return std::move(prop);
++            return prop;
+         }
+ 
+     public:
+@@ -476,7 +476,7 @@ class registration::bind<detail::prop, C
+                                                              getter_policy, setter_policy,
+                                                              Metadata_Count>>(name, type::get<Class_Type>(),
+                                                                               getter, setter, std::move(metadata_list));
+-            return std::move(prop);
++            return prop;
+         }
+ 
+     public:
+@@ -566,7 +566,7 @@ class registration::bind<detail::prop_re
+                                                              getter_policy, default_setter_policy, Metadata_Count>>(name, type::get<Class_Type>(),
+                                                                                                                     acc, std::move(metadata_list));
+ 
+-            return std::move(prop);
++            return prop;
+         }
+ 
+     public:
+@@ -659,7 +659,7 @@ class registration::bind<detail::meth, C
+                                                                      std::move(get_metadata(std::forward<Args>(args)...)),
+                                                                      std::move(get_default_args<type_list<Acc_Func>, function_type>(std::forward<Args>(args)...)),
+                                                                      std::move(create_param_infos<type_list<F>, function_type>(std::forward<Args>(args)...)) );
+-            return std::move(meth);
++            return meth;
+         }
+ 
+         template<typename Policy, std::size_t Metadata_Count, typename...TArgs, typename...Param_Args>
+@@ -770,7 +770,7 @@ class registration::bind<detail::enum_,
+                                                                                                 std::move(get_metadata(std::forward<Args>(args)...)));
+ 
+ 
+-            return std::move(enum_wrapper);
++            return enum_wrapper;
+         }
+ 
+     public:
diff --git a/rttr.changes b/rttr.changes
index 14a8561..dbe8ef7 100644
--- a/rttr.changes
+++ b/rttr.changes
@@ -1,3 +1,9 @@
+-------------------------------------------------------------------
+Tue Jun  4 02:54:17 UTC 2019 - Dan Weatherill <plasteredparrot@gmail.com>
+
+- remove_pessimizing_moves.patch: remove NRVO pessimizing moves
+  (fixes a warning on gcc 9 to fix current Factory build)
+
 -------------------------------------------------------------------
 Wed May  8 07:42:41 UTC 2019 - Andreas Schwab <schwab@suse.de>
 
diff --git a/rttr.spec b/rttr.spec
index f5f31cd..e2d2d36 100644
--- a/rttr.spec
+++ b/rttr.spec
@@ -36,6 +36,8 @@ Patch3:         skip_json_example.patch
 Patch4:         fix-include-permissions.patch
 # PATCH-FIX-UPSTREAM lp64.patch Support all LP64 architectures (https://github.com/rttrorg/rttr/pull/231)
 Patch5:         lp64.patch
+#PATCH-FIX-UPSTREAM remove_pessimizing_moves.patch remove pessimizing move calls (https://github.com/rttrorg/rttr/pull/243)
+Patch6:         remove_pessimizing_moves.patch 
 BuildRequires:  cmake
 BuildRequires:  dos2unix
 BuildRequires:  doxygen
@@ -56,6 +58,7 @@ library itself, which is written in C++.
 %patch3 -p1
 %patch4 -p1
 %patch5 -p1
+%patch6 -1p
 
 %build
 find . -type f -exec chmod a-x "{}" +