Comment updates

[builder.git] / source / target.h

diff --git a/source/target.h b/source/target.h
index d0e68f50588da69decb51b556dd633be63529c05..03b87557873e111a2b0eb986b90fb97b1cbe959a 100644 (file)
--- a/source/target.h
+++ b/source/target.h
@@ -45,7 +45,6 @@ protected:
        std::string install_location;
 
        Dependencies depends;
-       bool deps_ready;
 
        Target(Builder &, const std::string &);
 public:
@@ -56,19 +55,16 @@ public:
        const Package *get_package() const { return package; }
        const Component *get_component() const { return component; }
 
-       /**
-       Tries to locate a target that will help getting this target built.  If all
-       dependencies are up-to-date, returns this target.  If there are no targets
-       ready to be built (maybe because they are being built right now), returns 0.
-       */
+       /** Tries to locate a target that will help getting this target built.  If
+       all dependencies are up-to-date, returns this target.  If there are no
+       targets ready to be built (maybe because they are being built right now),
+       returns 0. */
        virtual Target *get_buildable_target();
 
-       /**
-       If this target is a proxy for another (such as Install or Symlink), return
-       that target.  Otherwise, return the target itself.
+       /** If this target is a proxy for another (such as Install), return that
+       target.  Otherwise, return the target itself.
 
-       Implementors should call the function recursively to find the final target.
-       */
+       Implementors should call the function recursively to find the final target. */
        virtual Target *get_real_target() { return this; }
 
        void set_tool(const Tool &);
@@ -81,36 +77,23 @@ public:
        const std::string &get_install_location() const { return install_location; }
        void add_depend(Target *);
        const Dependencies &get_depends() const { return depends; }
-       bool get_depends_ready() const { return deps_ready; }
-
-       /**
-       Finds dependencies for the target.  When all dependencies have been found,
-       the function should set deps_ready to true.
-       */
-       virtual void find_depends() { deps_ready = true; }
-
-       /**
-       Prepares the target by recursively preparing dependencies, then checking
-       whether rebuilding is needed.  A flag is used to prevent unnecessary
-       executions.
-       */
+
+       /** Finds dependencies for the target. */
+       virtual void find_depends() { }
+
+       /** Prepares the target by finding dependencies, recursively preparing them
+       and then checking whether rebuilding is needed. */
        virtual void prepare();
 
-       /**
-       Starts building the target.  Returns the Action used for building.
-       */
+       /** Starts building the target.  Returns the Action used for building. */
        Task *build();
 protected:
        void mark_rebuild(const std::string &);
 
-       /**
-       Checks if the target needs to be rebuilt and why.
-       */
+       /** Checks if the target needs to be rebuilt and why. */
        virtual void check_rebuild() = 0;
 
-       /**
-       Handler for the build_finished signal of Task.
-       */
+       /** Handler for Task::signal_finished. */
        virtual void build_finished(bool);
 };