rename basename to zimName, new member tmpFileName

openzim · Feb 23, 2021 · 8499a83 · 8499a83
1 parent d7ebc40
commit 8499a83
Show file tree

Hide file tree

Showing 3 changed files with 9 additions and 8 deletions.
diff --git a/src/writer/creator.cpp b/src/writer/creator.cpp
@@ -286,7 +286,7 @@ namespace zim
       ::close(data->out_fd);
 
       TINFO("rename tmpfile to final one.");
-      DEFAULTFS::rename(data->basename+".tmp", data->basename);
+      DEFAULTFS::rename(data->tmpFileName, data->zimName);
 
       TINFO("finish");
     }
@@ -405,19 +405,19 @@ namespace zim
         nbCompClusters(0),
         nbUnCompClusters(0),
         start_time(time(NULL)),
-        basename(fname)
+        zimName(fname),
+        tmpFileName(fname + ".tmp")
     {
-      auto zim_name = basename + ".tmp";
 #ifdef _WIN32
 int mode =  _S_IREAD | _S_IWRITE;
 #else
       mode_t mode = S_IRUSR | S_IWUSR | S_IRGRP | S_IROTH;
 #endif
-      out_fd = open(zim_name.c_str(), O_RDWR|O_CREAT|O_TRUNC, mode);
+      out_fd = open(tmpFileName.c_str(), O_RDWR|O_CREAT|O_TRUNC, mode);
       if (out_fd == -1){
         perror(nullptr);
         std::ostringstream ss;
-        ss << "Cannot create file " << zim_name;
+        ss << "Cannot create file " << tmpFileName;
         throw std::runtime_error(ss.str());
       }
       if(lseek(out_fd, CLUSTER_BASE_OFFSET, SEEK_SET) != CLUSTER_BASE_OFFSET) {

diff --git a/src/writer/creatordata.h b/src/writer/creatordata.h
@@ -98,7 +98,8 @@ namespace zim
         ThreadList workerThreads;
         std::thread  writerThread;
         const CompressionType compression;
-        std::string basename;
+        std::string zimName;
+        std::string tmpFileName;
         bool isEmpty = true;
         bool isExtended = false;
         zsize_t clustersSize;

diff --git a/src/writer/xapianHandler.cpp b/src/writer/xapianHandler.cpp
@@ -27,7 +27,7 @@
 using namespace zim::writer;
 
 FullTextXapianHandler::FullTextXapianHandler(CreatorData* data)
-  : mp_indexer(new XapianIndexer(data->basename+"_fulltext.idx", data->indexingLanguage, IndexingMode::FULL, true)),
+  : mp_indexer(new XapianIndexer(data->zimName+"_fulltext.idx", data->indexingLanguage, IndexingMode::FULL, true)),
     mp_creatorData(data)
 {}
 
@@ -71,7 +71,7 @@ void FullTextXapianHandler::handle(Dirent* dirent, std::shared_ptr<Item> item)
 }
 
 TitleXapianHandler::TitleXapianHandler(CreatorData* data)
-  : mp_indexer(new XapianIndexer(data->basename+"_title.idx", data->indexingLanguage, IndexingMode::TITLE, true)),
+  : mp_indexer(new XapianIndexer(data->zimName+"_title.idx", data->indexingLanguage, IndexingMode::TITLE, true)),
     mp_creatorData(data)
 {}