Remove the `visit` machinery in `RealisedPath`

In addition to being some ugly template trickery, it was also totally
useless as it was used in only one place where I could replace it by
just a few extra characters

2 files changed, 1 insertions, 14 deletions

diff --git a/src/libstore/realisation.cc b/src/libstore/realisation.cc
index e4276c040..cd74af4ee 100644
--- a/src/libstore/realisation.cc
+++ b/src/libstore/realisation.cc
@@ -47,7 +47,7 @@ Realisation Realisation::fromJSON(
 }
 
 StorePath RealisedPath::path() const {
-    return visit([](auto && arg) { return arg.getPath(); });
+    return std::visit([](auto && arg) { return arg.getPath(); }, raw);
 }
 
 void RealisedPath::closure(
diff --git a/src/libstore/realisation.hh b/src/libstore/realisation.hh
index 557f54362..7c91d802a 100644
--- a/src/libstore/realisation.hh
+++ b/src/libstore/realisation.hh
@@ -59,19 +59,6 @@ struct RealisedPath {
     RealisedPath(Realisation r) : raw(r) {}
 
     /**
-     * Syntactic sugar to run `std::visit` on the raw value:
-     * path.visit(blah) == std::visit(blah, path.raw)
-     */
-    template <class Visitor>
-    constexpr decltype(auto) visit(Visitor && vis) {
-        return std::visit(vis, raw);
-    }
-    template <class Visitor>
-    constexpr decltype(auto) visit(Visitor && vis) const {
-        return std::visit(vis, raw);
-    }
-
-    /**
      * Get the raw store path associated to this
      */
     StorePath path() const;