bug-hurd
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [PATCH,HURD] mach: compliance fixes for nanosleep


From: Roland McGrath
Subject: Re: [PATCH,HURD] mach: compliance fixes for nanosleep
Date: Fri, 20 Jul 2012 10:37:28 -0700 (PDT)

In a log entry, write a local variable name as FOO rather than `foo'.

Write 'remaining != NULL' in tests.  Fix the existing cases while you're
there.

If the second __gettimeofday calls, return failure as it did before.

Since I've just seen it repeated here and in your select change, it seems
worthwhile to add internal inlines for converting timeval/timespec to
mach_msg_timeout_t with the EINVAL check.  So if you're looking for more
cleanup to do, that's a worthwhile thing to pursue.  But you don't have to
do that before completing this change unless you want to.


Thanks,
Roland



reply via email to

[Prev in Thread] Current Thread [Next in Thread]