This is the mail archive of the
libc-alpha@sourceware.org
mailing list for the glibc project.
Re: [PATCH 1/2] support/timespec.h: Correct confusing comment
- From: Adhemerval Zanella <adhemerval dot zanella at linaro dot org>
- To: libc-alpha at sourceware dot org
- Date: Mon, 20 May 2019 15:42:11 -0300
- Subject: Re: [PATCH 1/2] support/timespec.h: Correct confusing comment
- References: <20190516122026.14205-1-mac@mcrowe.com>
On 16/05/2019 09:20, Mike Crowe wrote:
> I seem to have badly copied and pasted the comment describing
> TEST_TIMESPEC_EQUAL_OR_AFTER in such a way that it made no sense at all.
>
> * support/timespec.h: Correct confusing comment.
LTGM, I pushed for you.
> ---
> ChangeLog | 4 ++++
> support/timespec.h | 4 ++--
> 2 files changed, 6 insertions(+), 2 deletions(-)
>
> diff --git a/ChangeLog b/ChangeLog
> index ea7b3d4f48..f870deca1a 100644
> --- a/ChangeLog
> +++ b/ChangeLog
> @@ -1,3 +1,7 @@
> +2019-05-16 Mike Crowe <mac@mcrowe.com>
> +
> + * support/timespec.h: Correct confusing comment.
> +
> 2019-05-15 Mark Wielaard <mark@klomp.org>
>
> [BZ# 24476]
> diff --git a/support/timespec.h b/support/timespec.h
> index 4a8b341c5a..e9f466b3d1 100644
> --- a/support/timespec.h
> +++ b/support/timespec.h
> @@ -62,8 +62,8 @@ void test_timespec_equal_or_after_impl (const char *file, int line,
> errno = saved_errno; \
> })
>
> -/* Check that the timespec on the left represents a after before the
> - time on the right. */
> +/* Check that the timespec on the left represents a time equal to or
> + after the time on the right. */
> #define TEST_TIMESPEC_EQUAL_OR_AFTER(left, right) \
> test_timespec_equal_or_after_impl (__FILE__, __LINE__, left, right)
>
>