Replace src/csync/std/c_time.cpp with std::filesystem::last_write_time

src/csync/CMakeLists.txt

@@ -6,9 +6,6 @@ include(../common/common.cmake)
 set(csync_SRCS
   csync.cpp
   csync_exclude.cpp
-
-  std/c_time.cpp
-
 )
 
 if (WIN32)

src/libsync/filesystem.cpp

@@ -23,13 +23,14 @@
 
 #include "csync.h"
 #include "vio/csync_vio_local.h"
-#include "std/c_time.h"
 
 #ifdef Q_OS_WIN32
 #include "common/utility_win.h"
 #include <winsock2.h>
 #endif
 
+#include <filesystem>
+
 namespace OCC {
 
 bool FileSystem::fileEquals(const QString &fn1, const QString &fn2)
@@ -62,28 +63,25 @@ bool FileSystem::fileEquals(const QString &fn1, const QString &fn2)
 
 time_t FileSystem::getModTime(const QString &filename)
 {
-    csync_file_stat_t stat;
-    qint64 result = -1;
-    if (csync_vio_local_stat(filename, &stat) != -1
-        && (stat.modtime != 0)) {
-        result = stat.modtime;
-    } else {
-        result = Utility::qDateTimeToTime_t(QFileInfo(filename).lastModified());
-        qCWarning(lcFileSystem) << "Could not get modification time for" << filename
-                                << "with csync, using QFileInfo:" << result;
+    std::error_code error;
+    const auto time = std::filesystem::last_write_time(toFilesystemPath(filename), error);
+    const auto systemTime = std::chrono::system_clock::now() + (time - std::filesystem::file_time_type::clock::now());
+    if (error) {
+        qCWarning(lcFileSystem) << "Could not get modification time for" << filename << error.value() << QString::fromStdString(error.message());
+        Q_ASSERT(false);
+        return {};
     }
-    return result;
+    return std::chrono::system_clock::to_time_t(systemTime);
 }
 
 bool FileSystem::setModTime(const QString &filename, time_t modTime)
 {
-    struct timeval times[2];
-    times[0].tv_sec = times[1].tv_sec = modTime;
-    times[0].tv_usec = times[1].tv_usec = 0;
-    int rc = c_utimes(filename, times);
-    if (rc != 0) {
-        qCWarning(lcFileSystem) << "Error setting mtime for" << filename
-                                << "failed: rc" << rc << ", error message:" << strerror(errno);
+    const auto time = std::filesystem::file_time_type::clock::now() + (std::chrono::system_clock::from_time_t(modTime) - std::chrono::system_clock::now());
+    std::error_code error;
+    std::filesystem::last_write_time(toFilesystemPath(filename), time, error);
+    if (error) {
+        qCWarning(lcFileSystem) << "Error setting mtime for" << filename << "failed: rc" << error.value()
+                                << ", errno:" << QString::fromStdString(error.message());
         Q_ASSERT(false);
         return false;
     }

src/libsync/filesystem.h

@@ -22,6 +22,8 @@
 // Chain in the base include and extend the namespace
 #include "common/filesystembase.h"
 
+#include <filesystem>
+
 class QFile;
 
 namespace OCC {
@@ -38,6 +40,12 @@ class SyncJournal;
  */
 namespace FileSystem {
 
+    // https://github.com/qt/qtbase/blob/067b53864112c084587fa9a507eb4bde3d50a6e1/src/corelib/io/qfile.h#L40
+    inline std::filesystem::path toFilesystemPath(const QString &path)
+    {
+        return std::filesystem::path(reinterpret_cast<const char16_t *>(path.cbegin()), reinterpret_cast<const char16_t *>(path.cend()));
+    }
+
     /**
      * @brief compare two files with given filename and return true if they have the same content
      */