[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: NULL struct to futimes does not return current date/time on GNU/Hurd
From: |
Samuel Thibault |
Subject: |
Re: NULL struct to futimes does not return current date/time on GNU/Hurd |
Date: |
Sat, 23 Sep 2006 14:41:09 +0200 |
User-agent: |
Mutt/1.5.12-2006-07-14 |
Hi,
Thomas Schwinge, le Sat 23 Sep 2006 12:48:56 +0200, a écrit :
> + new_atime = *(time_value_t *) &tvp[0];
> + new_mtime = *(time_value_t *) &tvp[1];
Mmm, this might break sometime for whatever optimizing compiler reason
too. Why not stop using bogus casts, and rather use:
Index: sysdeps/mach/hurd/futimes.c
===================================================================
RCS file: /cvs/glibc/libc/sysdeps/mach/hurd/futimes.c,v
retrieving revision 1.1
diff -u -p -r1.1 futimes.c
--- sysdeps/mach/hurd/futimes.c 27 Aug 2002 02:09:20 -0000 1.1
+++ sysdeps/mach/hurd/futimes.c 23 Sep 2006 12:29:52 -0000
@@ -28,20 +28,22 @@
int
__futimes (int fd, const struct timeval tvp[2])
{
- struct timeval timevals[2];
error_t err;
+ time_value_t new_atime, new_mtime;
if (tvp == NULL)
- {
/* Setting the number of microseconds to `-1' tells the
underlying filesystems to use the current time. */
- timevals[1].tv_usec = timevals[0].tv_usec = (time_t)-1;
- tvp = timevals;
+ new_atime.microseconds = new_mtime.microseconds = -1;
+ else
+ {
+ new_atime.seconds = tvp[0].tv_sec;
+ new_atime.microseconds = tvp[0].tv_usec;
+ new_mtime.seconds = tvp[1].tv_sec;
+ new_mtime.microseconds = tvp[1].tv_usec;
}
- err = HURD_DPORT_USE (fd, __file_utimes (port,
- *(time_value_t *) &tvp[0],
- *(time_value_t *) &tvp[1]));
+ err = HURD_DPORT_USE (fd, __file_utimes (port, new_atime, new_mtime));
return err ? __hurd_dfail (fd, err) : 0;
}
weak_alias (__futimes, futimes)
which just precisely explains the compiler what we want, and nothing
more, nothing less.
Samuel