Re: [PATCH 2/2] comments: avoid using the gender of our users

From: Derrick Stolee <stolee@gmail.com>
Date: Jun 15, 2021, 13:31 UTC
Message-ID: <94ce6d7d-f923-edb3-12f5-4b11149c9f6f@gmail.com>
In-Reply-To: <20210611202819.47077-3-felipe.contreras@gmail.com>

On 6/11/2021 4:28 PM, Felipe Contreras wrote:

> We generally avoid specifying the gender of our users in order to be
> more politically correct, but sometimes due to lack of time and/or
> creativity a few slip by.

s/politically correct/inclusive/

Instead of "lack of time and/or creativity" perhaps be more generous to our fellow contributors and instead say...

  but sometimes a few slip by due to habit.

Show 6 quoted lines

> @@ -908,7 +908,7 @@ int parse_expiry_date(const char *date, timestamp_t *timestamp)
>  		/*
>  		 * We take over "now" here, which usually translates
>  		 * to the current timestamp.  This is because the user
> -		 * really means to expire everything she has done in
> +		 * really means to expire everything that has been done in

nit: s/has been/was/

We can be a little more active here.

-Stolee