Optimize apr_file_mtime_set() on Windows.

* file_io/win32/filestat.c
  (apr_file_mtime_set): Pass NULL as ATIME and CTIME when calling SetFileTime()
   to prevent changing ATIME and CTIME instead of retrieving these times using
   GetFileTime().

git-svn-id: https://svn.apache.org/repos/asf/apr/apr/trunk@1896623 13f79535-47bb-0310-9956-ffa450edef68

2 files changed, 4 insertions, 11 deletions

diff --git a/CHANGES b/CHANGES
index ee78b60d7..a7b48431c 100644
--- a/CHANGES
+++ b/CHANGES
@@ -258,6 +258,8 @@ Changes for APR 2.0.0
   *) apr_proc_create(): Properly escape arguments containing whitespace
      characters on Windows [Ivan Zhakov]
 
+  *) apr_file_mtime_set(): Minor optimization on Windows. [Ivan Zhakov]
+
 Changes for APR and APR-util 1.7.x and later:
 
   *) http://svn.apache.org/viewvc/apr/apr/branches/1.7.x/CHANGES?view=markup
diff --git a/file_io/win32/filestat.c b/file_io/win32/filestat.c
index ce32cf91d..0138341dc 100644
--- a/file_io/win32/filestat.c
+++ b/file_io/win32/filestat.c
@@ -752,20 +752,11 @@ APR_DECLARE(apr_status_t) apr_file_mtime_set(const char *fname,
                        APR_FPROT_OS_DEFAULT, pool);
     if (!rv)
     {
-        FILETIME file_ctime;
-        FILETIME file_atime;
         FILETIME file_mtime;
 
-        if (!GetFileTime(thefile->filehand,
-                         &file_ctime, &file_atime, &file_mtime))
+        AprTimeToFileTime(&file_mtime, mtime);
+        if (!SetFileTime(thefile->filehand, NULL, NULL, &file_mtime))
             rv = apr_get_os_error();
-        else
-        {
-            AprTimeToFileTime(&file_mtime, mtime);
-            if (!SetFileTime(thefile->filehand,
-                             &file_ctime, &file_atime, &file_mtime))
-                rv = apr_get_os_error();
-        }
 
         apr_file_close(thefile);
     }