X-Git-Url: https://git.lyx.org/gitweb/?a=blobdiff_plain;f=src%2FConverterCache.cpp;h=76721835e61f271c181082ad7e3e04f0bf3c193f;hb=dc7c7ca03eca25cf7a6395c155b70dac1594e481;hp=813243f48b0ae47db60e43c92860f3fa3fe0b21b;hpb=b90919e48b7efaaabcab7013926654fb52b3d58f;p=lyx.git

diff --git a/src/ConverterCache.cpp b/src/ConverterCache.cpp
index 813243f48b..76721835e6 100644
--- a/src/ConverterCache.cpp
+++ b/src/ConverterCache.cpp
@@ -60,9 +60,9 @@ public:
 		: timestamp(t), checksum(c)
 	{
 		ostringstream os;
-		os << setw(10) << setfill('0') << do_crc(orig_from.absFilename())
+		os << setw(10) << setfill('0') << do_crc(orig_from.absFileName())
 		   << '-' << to_format;
-		cache_name = FileName(addName(cache_dir.absFilename(), os.str()));
+		cache_name = FileName(addName(cache_dir.absFileName(), os.str()));
 		LYXERR(Debug::FILES, "Add file cache item " << orig_from
 				     << ' ' << to_format << ' ' << cache_name
 				     << ' ' << long(timestamp) << ' ' << checksum << '.');
@@ -105,7 +105,7 @@ public:
 void ConverterCache::Impl::readIndex()
 {
 	time_t const now = current_time();
-	FileName const index(addName(cache_dir.absFilename(), "index"));
+	FileName const index(addName(cache_dir.absFileName(), "index"));
 	ifstream is(index.toFilesystemEncoding().c_str());
 	Lexer lex;
 	lex.setStream(is);
@@ -165,7 +165,7 @@ void ConverterCache::Impl::readIndex()
 
 void ConverterCache::Impl::writeIndex()
 {
-	FileName const index(addName(cache_dir.absFilename(), "index"));
+	FileName const index(addName(cache_dir.absFileName(), "index"));
 	ofstream os(index.toFilesystemEncoding().c_str());
 	os.close();
 	if (!index.changePermission(0600))
@@ -178,7 +178,7 @@ void ConverterCache::Impl::writeIndex()
 		FormatCacheType::const_iterator it2 = format_cache.begin();
 		FormatCacheType::const_iterator const end2 = format_cache.end();
 		for (; it2 != end2; ++it2)
-			os << Lexer::quoteString(it1->first.absFilename())
+			os << Lexer::quoteString(it1->first.absFileName())
 			   << ' ' << it2->first << ' '
 			   << it2->second.timestamp << ' '
 			   << it2->second.checksum << '\n';
@@ -216,9 +216,6 @@ ConverterCache::ConverterCache()
 
 ConverterCache::~ConverterCache()
 {
-	if (!lyxrc.use_converter_cache)
-		return;
-	pimpl_->writeIndex();
 	delete pimpl_;
 }
 
@@ -237,7 +234,7 @@ void ConverterCache::init()
 		return;
 	// We do this here and not in the constructor because package() gets
 	// initialized after all static variables.
-	cache_dir = FileName(addName(package().user_support().absFilename(), "cache"));
+	cache_dir = FileName(addName(package().user_support().absFileName(), "cache"));
 	if (!cache_dir.exists())
 		if (!cache_dir.createDirectory(0700)) {
 			lyxerr << "Could not create cache directory `"
@@ -248,6 +245,15 @@ void ConverterCache::init()
 }
 
 
+void ConverterCache::writeIndex() const
+{
+	if (!lyxrc.use_converter_cache 
+		  || cache_dir.empty())
+		return;
+	pimpl_->writeIndex();
+}
+
+
 void ConverterCache::add(FileName const & orig_from, string const & to_format,
 		FileName const & converted_file) const
 {
@@ -259,10 +265,10 @@ void ConverterCache::add(FileName const & orig_from, string const & to_format,
 
 	// FIXME: Should not hardcode this (see bug 3819 for details)
 	if (to_format == "pstex") {
-		FileName const converted_eps(changeExtension(converted_file.absFilename(), "eps"));
+		FileName const converted_eps(changeExtension(converted_file.absFileName(), "eps"));
 		add(orig_from, "eps", converted_eps);
 	} else if (to_format == "pdftex") {
-		FileName const converted_pdf(changeExtension(converted_file.absFilename(), "pdf"));
+		FileName const converted_pdf(changeExtension(converted_file.absFileName(), "pdf"));
 		add(orig_from, "pdf", converted_pdf);
 	}
 
@@ -288,7 +294,7 @@ void ConverterCache::add(FileName const & orig_from, string const & to_format,
 		}
 		item->checksum = checksum;
 		if (!mover.copy(converted_file, item->cache_name,
-		              onlyFilename(item->cache_name.absFilename()))) {
+		              onlyFileName(item->cache_name.absFileName()))) {
 			LYXERR(Debug::FILES, "Could not copy file " << orig_from << " to "
 				<< item->cache_name);
 		} else if (!item->cache_name.changePermission(0600)) {
@@ -299,7 +305,7 @@ void ConverterCache::add(FileName const & orig_from, string const & to_format,
 		CacheItem new_item(orig_from, to_format, timestamp,
 				orig_from.checksum());
 		if (mover.copy(converted_file, new_item.cache_name,
-		              onlyFilename(new_item.cache_name.absFilename()))) {
+		              onlyFileName(new_item.cache_name.absFileName()))) {
 			if (!new_item.cache_name.changePermission(0600)) {
 				LYXERR(Debug::FILES, "Could not change file mode"
 					<< new_item.cache_name);
@@ -423,11 +429,11 @@ bool ConverterCache::copy(FileName const & orig_from, string const & to_format,
 
 	// FIXME: Should not hardcode this (see bug 3819 for details)
 	if (to_format == "pstex") {
-		FileName const dest_eps(changeExtension(dest.absFilename(), "eps"));
+		FileName const dest_eps(changeExtension(dest.absFileName(), "eps"));
 		if (!copy(orig_from, "eps", dest_eps))
 			return false;
 	} else if (to_format == "pdftex") {
-		FileName const dest_pdf(changeExtension(dest.absFilename(), "pdf"));
+		FileName const dest_pdf(changeExtension(dest.absFileName(), "pdf"));
 		if (!copy(orig_from, "pdf", dest_pdf))
 			return false;
 	}
@@ -436,7 +442,7 @@ bool ConverterCache::copy(FileName const & orig_from, string const & to_format,
 	LASSERT(item, /**/);
 	Mover const & mover = getMover(to_format);
 	return mover.copy(item->cache_name, dest,
-	                  onlyFilename(dest.absFilename()));
+	                  onlyFileName(dest.absFileName()));
 }
 
 } // namespace lyx