Make PartialComponent non-movable. This catches at compile-time some cases where PartialComponent is used for non-temporary values. See issue 67.

1 files changed, 1 insertions, 2 deletions

diff --git a/include/fruit/component.h b/include/fruit/component.h
index 5e2a4ec..5fbfdc6 100644
--- a/include/fruit/component.h
+++ b/include/fruit/component.h
@@ -143,8 +143,6 @@ PartialComponent<> createComponent();
 template <typename... Bindings>
 class PartialComponent {
 public:
-  PartialComponent(PartialComponent&&) = default;
-
   PartialComponent& operator=(PartialComponent&&) = delete;
   PartialComponent& operator=(const PartialComponent&) = delete;
 
@@ -1035,6 +1033,7 @@ private:
 
   // Do not use. Only use PartialComponent for temporaries, and then convert it to a Component.
   PartialComponent(const PartialComponent&) = delete;
+  PartialComponent(PartialComponent&&) = delete;
 
   PartialComponent(fruit::impl::PartialComponentStorage<Bindings...> storage);