Skip to content

Commit 45d38aa

Browse files
committed
Win32: implement nanosecond-precision file times
We no longer use any of MSVCRT's stat-functions, so there's no need to stick to a CRT-compatible 'struct stat' either. Define and use our own POSIX-2013-compatible 'struct stat' with nanosecond- precision file times. Note: This can cause performance issues when using Git variants with different file time resolutions, as the timestamps are stored in the Git index: after updating the index with a Git variant that uses second-precision file times, a nanosecond-aware Git will think that pretty much every single file listed in the index is out of date. Signed-off-by: Karsten Blees <[email protected]> Signed-off-by: Johannes Schindelin <[email protected]>
1 parent 34d51bc commit 45d38aa

File tree

3 files changed

+36
-20
lines changed

3 files changed

+36
-20
lines changed

compat/mingw.c

Lines changed: 10 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -562,9 +562,11 @@ static inline long long filetime_to_hnsec(const FILETIME *ft)
562562
return winTime - 116444736000000000LL;
563563
}
564564

565-
static inline time_t filetime_to_time_t(const FILETIME *ft)
565+
static inline void filetime_to_timespec(const FILETIME *ft, struct timespec *ts)
566566
{
567-
return (time_t)(filetime_to_hnsec(ft) / 10000000);
567+
long long hnsec = filetime_to_hnsec(ft);
568+
ts->tv_sec = (time_t)(hnsec / 10000000);
569+
ts->tv_nsec = (hnsec % 10000000) * 100;
568570
}
569571

570572
/**
@@ -623,9 +625,9 @@ static int do_lstat(int follow, const char *file_name, struct stat *buf)
623625
buf->st_size = fdata.nFileSizeLow |
624626
(((off_t)fdata.nFileSizeHigh)<<32);
625627
buf->st_dev = buf->st_rdev = 0; /* not used by Git */
626-
buf->st_atime = filetime_to_time_t(&(fdata.ftLastAccessTime));
627-
buf->st_mtime = filetime_to_time_t(&(fdata.ftLastWriteTime));
628-
buf->st_ctime = filetime_to_time_t(&(fdata.ftCreationTime));
628+
filetime_to_timespec(&(fdata.ftLastAccessTime), &(buf->st_atim));
629+
filetime_to_timespec(&(fdata.ftLastWriteTime), &(buf->st_mtim));
630+
filetime_to_timespec(&(fdata.ftCreationTime), &(buf->st_ctim));
629631
if (fdata.dwFileAttributes & FILE_ATTRIBUTE_REPARSE_POINT) {
630632
WIN32_FIND_DATAW findbuf;
631633
HANDLE handle = FindFirstFileW(wfilename, &findbuf);
@@ -723,9 +725,9 @@ static int get_file_info_by_handle(HANDLE hnd, struct stat *buf)
723725
buf->st_size = fdata.nFileSizeLow |
724726
(((off_t)fdata.nFileSizeHigh)<<32);
725727
buf->st_dev = buf->st_rdev = 0; /* not used by Git */
726-
buf->st_atime = filetime_to_time_t(&(fdata.ftLastAccessTime));
727-
buf->st_mtime = filetime_to_time_t(&(fdata.ftLastWriteTime));
728-
buf->st_ctime = filetime_to_time_t(&(fdata.ftCreationTime));
728+
filetime_to_timespec(&(fdata.ftLastAccessTime), &(buf->st_atim));
729+
filetime_to_timespec(&(fdata.ftLastWriteTime), &(buf->st_mtim));
730+
filetime_to_timespec(&(fdata.ftCreationTime), &(buf->st_ctim));
729731
return 0;
730732
}
731733

compat/mingw.h

Lines changed: 26 additions & 10 deletions
Original file line numberDiff line numberDiff line change
@@ -327,18 +327,41 @@ static inline int getrlimit(int resource, struct rlimit *rlp)
327327
}
328328

329329
/*
330-
* Use mingw specific stat()/lstat()/fstat() implementations on Windows.
330+
* Use mingw specific stat()/lstat()/fstat() implementations on Windows,
331+
* including our own struct stat with 64 bit st_size and nanosecond-precision
332+
* file times.
331333
*/
332334
#ifndef __MINGW64_VERSION_MAJOR
333335
#define off_t off64_t
334336
#define lseek _lseeki64
337+
struct timespec {
338+
time_t tv_sec;
339+
long tv_nsec;
340+
};
335341
#endif
336342

337-
/* use struct stat with 64 bit st_size */
343+
struct mingw_stat {
344+
_dev_t st_dev;
345+
_ino_t st_ino;
346+
_mode_t st_mode;
347+
short st_nlink;
348+
short st_uid;
349+
short st_gid;
350+
_dev_t st_rdev;
351+
off64_t st_size;
352+
struct timespec st_atim;
353+
struct timespec st_mtim;
354+
struct timespec st_ctim;
355+
};
356+
357+
#define st_atime st_atim.tv_sec
358+
#define st_mtime st_mtim.tv_sec
359+
#define st_ctime st_ctim.tv_sec
360+
338361
#ifdef stat
339362
#undef stat
340363
#endif
341-
#define stat _stati64
364+
#define stat mingw_stat
342365
int mingw_lstat(const char *file_name, struct stat *buf);
343366
int mingw_stat(const char *file_name, struct stat *buf);
344367
int mingw_fstat(int fd, struct stat *buf);
@@ -351,13 +374,6 @@ int mingw_fstat(int fd, struct stat *buf);
351374
#endif
352375
#define lstat mingw_lstat
353376

354-
#ifndef _stati64
355-
# define _stati64(x,y) mingw_stat(x,y)
356-
#elif defined (_USE_32BIT_TIME_T)
357-
# define _stat32i64(x,y) mingw_stat(x,y)
358-
#else
359-
# define _stat64(x,y) mingw_stat(x,y)
360-
#endif
361377

362378
int mingw_utime(const char *file_name, const struct utimbuf *times);
363379
#define utime mingw_utime

config.mak.uname

Lines changed: 0 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -370,7 +370,6 @@ ifeq ($(uname_S),Windows)
370370
RUNTIME_PREFIX = YesPlease
371371
HAVE_WPGMPTR = YesWeDo
372372
NO_ST_BLOCKS_IN_STRUCT_STAT = YesPlease
373-
NO_NSEC = YesPlease
374373
USE_WIN32_MMAP = YesPlease
375374
MMAP_PREVENTS_DELETE = UnfortunatelyYes
376375
# USE_NED_ALLOCATOR = YesPlease
@@ -520,7 +519,6 @@ ifneq (,$(findstring MINGW,$(uname_S)))
520519
RUNTIME_PREFIX = YesPlease
521520
HAVE_WPGMPTR = YesWeDo
522521
NO_ST_BLOCKS_IN_STRUCT_STAT = YesPlease
523-
NO_NSEC = YesPlease
524522
USE_WIN32_MMAP = YesPlease
525523
MMAP_PREVENTS_DELETE = UnfortunatelyYes
526524
USE_NED_ALLOCATOR = YesPlease

0 commit comments

Comments
 (0)