From f0988148767b92838203bd63db2c95b4a0add8bf Mon Sep 17 00:00:00 2001 From: Christian Ebner Date: Tue, 4 Mar 2025 19:35:40 +0100 Subject: [PATCH] datastore: use libc's timespec constants instead of redefinition Use the UTIME_NOW and UTIME_OMIT constants defined in libc crate instead of redefining them. This improves consistency, as utimesat and its timespec parameter are also defined via the libc crate. Signed-off-by: Christian Ebner --- pbs-datastore/src/chunk_store.rs | 7 ++----- 1 file changed, 2 insertions(+), 5 deletions(-) diff --git a/pbs-datastore/src/chunk_store.rs b/pbs-datastore/src/chunk_store.rs index 29d5874a..5e02909a 100644 --- a/pbs-datastore/src/chunk_store.rs +++ b/pbs-datastore/src/chunk_store.rs @@ -220,19 +220,16 @@ impl ChunkStore { // unwrap: only `None` in unit tests assert!(self.locker.is_some()); - const UTIME_NOW: i64 = (1 << 30) - 1; - const UTIME_OMIT: i64 = (1 << 30) - 2; - let times: [libc::timespec; 2] = [ // access time -> update to now libc::timespec { tv_sec: 0, - tv_nsec: UTIME_NOW, + tv_nsec: libc::UTIME_NOW, }, // modification time -> keep as is libc::timespec { tv_sec: 0, - tv_nsec: UTIME_OMIT, + tv_nsec: libc::UTIME_OMIT, }, ];