X-Git-Url: http://git.tdb.fi/?a=blobdiff_plain;f=source%2Fcsourcefile.cpp;h=ca061d2b19e581ccdc1f6f308c2848533f8135c9;hb=7606161ee4a6fd13c2f5ddca413ba3582df66f03;hp=2162cf0873ee1f7b9fd9a43a82649c2b47a4ba05;hpb=7e5ac6af8987bf12f3e338d00e96e8cb74f3534b;p=builder.git

diff --git a/source/csourcefile.cpp b/source/csourcefile.cpp
index 2162cf0..ca061d2 100644
--- a/source/csourcefile.cpp
+++ b/source/csourcefile.cpp
@@ -11,76 +11,58 @@ using namespace std;
 using namespace Msp;
 
 CSourceFile::CSourceFile(Builder &b, const FS::Path &p):
-	SourceFile(b, 0, p)
+	SourceFile(b, p)
 { }
 
 CSourceFile::CSourceFile(Builder &b, const Component &c, const FS::Path &p):
-	SourceFile(b, &c, p)
+	SourceFile(b, c, p)
 {
 	string ext = FS::extpart(FS::basename(path));
 	if(ext==".h" || ext==".H" || ext==".hpp")
-		install_location = "include/"+comp->get_name();
+		install_location = FS::Path("include")/package->get_name();
 }
 
-void CSourceFile::find_depends()
+void CSourceFile::find_dependencies()
 {
-	if(!comp)
-	{
-		deps_ready = true;
+	if(!component || !mtime)
 		return;
-	}
 
-	const SourcePackage &spkg = comp->get_package();
-	string relname = FS::relative(path, spkg.get_source()).str();
-	DependencyCache &deps_cache = spkg.get_deps_cache();
-	bool deps_found = false;
-	if(mtime<deps_cache.get_mtime())
-	{
-		try
-		{
-			includes = deps_cache.get_deps(relname);
-			deps_found = true;
-		}
-		catch(const key_error &)
-		{ }
-	}
+	const SourcePackage &spkg = component->get_package();
 
-	if(!deps_found)
+	Cache &cache = spkg.get_cache();
+	if(mtime<cache.get_mtime() && cache.has_key(this, "includes"))
+		includes = cache.get_values(this, "includes");
+	else
 	{
-		try
-		{
-			IO::BufferedFile in(path.str());
+		IO::BufferedFile in(path.str());
 
-			if(builder.get_verbose()>=4)
-				IO::print("Reading includes from %s\n", path.str());
+		builder.get_logger().log("files", format("Reading includes from %s", path.str()));
 
-			Regex r_include("^[ \t]*#include[ \t]+([\"<].*)[\">]");
+		Regex r_include("^[ \t]*#include[ \t]+([\"<].*)[\">]");
 
-			string line;
-			while(in.getline(line))
-				if(RegMatch match = r_include.match(line))
-					includes.push_back(match[1].str);
+		string line;
+		while(in.getline(line))
+			if(RegMatch match = r_include.match(line))
+				includes.push_back(match[1].str);
 
-			deps_cache.set_deps(relname, includes);
-		}
-		catch(const IO::file_not_found &)
-		{
-			if(builder.get_verbose()>=4)
-				IO::print("Failed to read includes from %s\n", path.str());
-			deps_ready = true;
-			return;
-		}
+		cache.set_values(this, "includes", includes);
 	}
 
-	const StringList &incpath = comp->get_build_info().incpath;
+	const BuildInfo::PathList &incpath = component->get_build_info().incpath;
+	BuildInfo::PathList local_incpath = incpath;
+	local_incpath.push_front(FS::dirname(path).str());
 
-	FS::Path dir = FS::dirname(path);
-	for(list<string>::iterator i=includes.begin(); i!=includes.end(); ++i)
+	for(IncludeList::iterator i=includes.begin(); i!=includes.end(); ++i)
 	{
-		Target *hdr = builder.get_vfs().find_header(*i, dir, incpath);
+		Target *hdr = builder.get_vfs().find_header(i->substr(1), ((*i)[0]=='"' ? local_incpath : incpath));
 		if(hdr)
-			add_depend(hdr);
+			add_dependency(*hdr);
 	}
+}
 
-	deps_ready = true;
+void CSourceFile::modified()
+{
+	includes.clear();
+	depends.clear();
+	find_dependencies();
 }