Rearrange Target members

[builder.git] / source / objectfile.cpp

diff --git a/source/objectfile.cpp b/source/objectfile.cpp
index 39372a415c0e67350d0a4862282a74e489dd213e..7aafe90d68c1412594a086b998cb828d78605269 100644 (file)
--- a/source/objectfile.cpp
+++ b/source/objectfile.cpp
@@ -10,11 +10,23 @@ using namespace std;
 using namespace Msp;
 
 ObjectFile::ObjectFile(Builder &b, const Component &c, SourceFile &s):
-       FileTarget(b, &c.get_package(), generate_target_path(c, FS::relative(s.get_path(), c.get_package().get_source()).str())),
-       comp(c),
+       FileTarget(b, c.get_package(), generate_target_path(c, FS::relative(s.get_path(), c.get_package().get_source()).str())),
        source(s)       
 {
-       add_depend(&source);
+       component = &c;
+       add_depend(source);
+}
+
+FS::Path ObjectFile::generate_target_path(const Component &comp, const string &src)
+{
+       const SourcePackage &pkg = comp.get_package();
+       string fn = FS::basepart(src)+".o";
+       if(!fn.compare(0, 2, "./"))
+               fn.erase(0, 2);
+       for(string::iterator i=fn.begin(); i!=fn.end(); ++i)
+               if(*i=='/')
+                       *i = '_';
+       return pkg.get_temp_dir()/comp.get_name()/fn;
 }
 
 void ObjectFile::find_depends()
@@ -26,7 +38,6 @@ void ObjectFile::find_depends()
        }
 }
 
-
 void ObjectFile::find_depends(FileTarget *tgt)
 {
        FileTarget *rtgt = dynamic_cast<FileTarget *>(tgt->get_real_target());
@@ -42,7 +53,7 @@ void ObjectFile::find_depends(FileTarget *tgt)
        {
                /* The target has been displaced by installing it.  Displace any
                dependencies that come from the same package as well. */
-               const SourcePackage *tpkg = dynamic_cast<const SourcePackage *>(rtgt->get_package());
+               const SourcePackage *tpkg = rtgt->get_package();
                for(Dependencies::const_iterator i=tdeps.begin(); i!=tdeps.end(); ++i)
                {
                        FileTarget *file = dynamic_cast<FileTarget *>(*i);
@@ -59,17 +70,5 @@ void ObjectFile::find_depends(FileTarget *tgt)
 
        for(Dependencies::const_iterator i=deps_to_add.begin(); i!=deps_to_add.end(); ++i)
                if(find(depends.begin(), depends.end(), *i)==depends.end())
-                       add_depend(*i);
-}
-
-FS::Path ObjectFile::generate_target_path(const Component &comp, const string &src)
-{
-       const SourcePackage &pkg = comp.get_package();
-       string fn = FS::basepart(src)+".o";
-       if(!fn.compare(0, 2, "./"))
-               fn.erase(0, 2);
-       for(string::iterator i=fn.begin(); i!=fn.end(); ++i)
-               if(*i=='/')
-                       *i = '_';
-       return pkg.get_temp_dir()/comp.get_name()/fn;
+                       add_depend(**i);
 }