diff options
| author | eldritch horrors <pennae@lix.systems> | 2024-07-09 23:14:46 +0200 |
|---|---|---|
| committer | eldritch horrors <pennae@lix.systems> | 2024-07-11 21:31:52 +0000 |
| commit | 4b109ec1a8fc4550150f56f0f46f2f41d844bda8 (patch) | |
| tree | 36e924ef4097521b7c8ba6aceacc427acb133819 /src/libstore/build/worker.hh | |
| parent | a5d431a9119c2560608768d9ec70fa37d5a6a985 (diff) | |
libstore: remove upcast_goal
upcast_goal was only ever needed to break circular includes, but the
same solution that gave us upcast_goal also lets us fully remove it:
just upcast goals without a wrapper function, but only in .cc files.
Change-Id: I9c71654b2535121459ba7dcfd6c5da5606904032
Diffstat (limited to 'src/libstore/build/worker.hh')
| -rw-r--r-- | src/libstore/build/worker.hh | 15 |
1 files changed, 0 insertions, 15 deletions
diff --git a/src/libstore/build/worker.hh b/src/libstore/build/worker.hh index ba4cd88d7..e1d8e5031 100644 --- a/src/libstore/build/worker.hh +++ b/src/libstore/build/worker.hh @@ -17,21 +17,6 @@ struct DerivationGoal; struct PathSubstitutionGoal; class DrvOutputSubstitutionGoal; -/** - * Workaround for not being able to declare a something like - * - * ```c++ - * class PathSubstitutionGoal : public Goal; - * ``` - * even when Goal is a complete type. - * - * This is still a static cast. The purpose of exporting it is to define it in - * a place where `PathSubstitutionGoal` is concrete, and use it in a place where it - * is opaque. - */ -GoalPtr upcast_goal(std::shared_ptr<PathSubstitutionGoal> subGoal); -GoalPtr upcast_goal(std::shared_ptr<DrvOutputSubstitutionGoal> subGoal); - typedef std::chrono::time_point<std::chrono::steady_clock> steady_time_point; /** |