Clean up ctors a bit

3 files changed, 12 insertions, 7 deletions

diff --git a/paludis/resolver/decider.cc b/paludis/resolver/decider.cc
index eda0b992b..f7d2c129d 100644
--- a/paludis/resolver/decider.cc
+++ b/paludis/resolver/decider.cc
@@ -1527,7 +1527,15 @@ Decider::_get_error_resolvents_for(
     DestinationTypes destination_types(_get_destination_types_for_error(spec, reason));
     for (EnumIterator<DestinationType> t, t_end(last_dt) ; t != t_end ; ++t)
         if (destination_types[*t])
-            result->push_back(Resolvent(spec, true, *t));
+        {
+            Resolvent resolvent(spec, make_named_values<SlotNameOrNull>(
+                        n::name_or_null() = make_null_shared_ptr(),
+                        n::null_means_unknown() = true
+                        ),
+                    *t);
+
+            result->push_back(resolvent);
+        }
     return result;
 }
 
diff --git a/paludis/resolver/resolvent.cc b/paludis/resolver/resolvent.cc
index 2a9586d41..1ed9426a6 100644
--- a/paludis/resolver/resolvent.cc
+++ b/paludis/resolver/resolvent.cc
@@ -93,14 +93,11 @@ Resolvent::Resolvent(
 
 Resolvent::Resolvent(
         const PackageDepSpec & spec,
-        const bool b,
+        const SlotNameOrNull & s,
         const DestinationType t) :
     destination_type(n::destination_type() = t),
     package(n::package() = *spec.package_ptr()),
-    slot(make_named_values<SlotNameOrNull>(
-                n::name_or_null() = make_null_shared_ptr(),
-                n::null_means_unknown() = b
-                ))
+    slot(s)
 {
 }
 
diff --git a/paludis/resolver/resolvent.hh b/paludis/resolver/resolvent.hh
index da86f1c3e..4ab1b6e61 100644
--- a/paludis/resolver/resolvent.hh
+++ b/paludis/resolver/resolvent.hh
@@ -54,8 +54,8 @@ namespace paludis
             Resolvent(const QualifiedPackageName &, const SlotName &, const DestinationType);
             Resolvent(const QualifiedPackageName &, const SlotNameOrNull &, const DestinationType);
 
-            Resolvent(const PackageDepSpec &, const bool, const DestinationType);
             Resolvent(const PackageDepSpec &, const SlotName &, const DestinationType);
+            Resolvent(const PackageDepSpec &, const SlotNameOrNull &, const DestinationType);
 
             Resolvent(const std::shared_ptr<const PackageID> &, const DestinationType);