# [PATCH] date.c: phrase a comment gender neutral 4 messages from 2014-11-18 to 2014-11-18. Participants: Stefan Beller, Jonathan Nieder, Junio C Hamano. Thread: https://gitlist.dev/t/38001 ## Stefan Beller, 2014-11-18 19:24 Subject: [PATCH] date.c: phrase a comment gender neutral Message-ID: <1416338668-30683-1-git-send-email-sbeller@google.com> URL: https://gitlist.dev/e/1416338668-30683-1-git-send-email-sbeller%40google.com ``` When looking for suitable functions to print dates, I found this gem. Let's make it gender neutral as nowadays people get upset by this gender debate. Signed-off-by: Stefan Beller --- date.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/date.c b/date.c index 59dfe57..19fb439 100644 --- a/date.c +++ b/date.c @@ -732,7 +732,7 @@ int parse_expiry_date(const char *date, unsigned long *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 they have done in * the past, and by definition reflogs are the record * of the past, and there is nothing from the future * to be kept. -- 2.2.0.rc2.5.gf7b9fb2 ``` ## Jonathan Nieder, 2014-11-18 19:42 Subject: Re: [PATCH] date.c: phrase a comment gender neutral Message-ID: <20141118194242.GJ6527@google.com> URL: https://gitlist.dev/e/20141118194242.GJ6527%40google.com In-Reply-To: <1416338668-30683-1-git-send-email-sbeller@google.com> ``` Stefan Beller wrote: > When looking for suitable functions to print dates, > I found this gem. Let's make it gender neutral as > nowadays people get upset by this gender debate. Eh, I'm not upset. Hope that helps, Jonathan ``` ## Junio C Hamano, 2014-11-18 20:03 Subject: Re: [PATCH] date.c: phrase a comment gender neutral Message-ID: URL: https://gitlist.dev/e/xmqq4mtwwa1n.fsf%40gitster.dls.corp.google.com In-Reply-To: <1416338668-30683-1-git-send-email-sbeller@google.com> ``` Stefan Beller writes: > When looking for suitable functions to print dates, > I found this gem. Let's make it gender neutral as > nowadays people get upset by this gender debate. For some time I used to use "she/her" on Mondays, Wednesdays and Fridays and "he/his" on other days to balance them, and you are seeing the artifact of that. Some people might feel that we would be better off using "they" all the time, but IMO it's such a minor thing that once it _is_ in the tree, it's not really worth the patch noise to go and fix it up. > > Signed-off-by: Stefan Beller > --- > date.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/date.c b/date.c > index 59dfe57..19fb439 100644 > --- a/date.c > +++ b/date.c > @@ -732,7 +732,7 @@ int parse_expiry_date(const char *date, unsigned long *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 they have done in > * the past, and by definition reflogs are the record > * of the past, and there is nothing from the future > * to be kept. ``` ## Stefan Beller, 2014-11-18 20:07 Subject: Re: [PATCH] date.c: phrase a comment gender neutral Message-ID: URL: https://gitlist.dev/e/CAGZ79kYtV3d0iTuzuh%3D%2BpzSOYJ8O3MUS4YFq2FbwS-pinU2%2BiA%40mail.gmail.com In-Reply-To: ``` ok. I'll stop sending such gender related nits. On Tue, Nov 18, 2014 at 12:03 PM, Junio C Hamano wrote: > Stefan Beller writes: > >> When looking for suitable functions to print dates, >> I found this gem. Let's make it gender neutral as >> nowadays people get upset by this gender debate. > > For some time I used to use "she/her" on Mondays, Wednesdays and > Fridays and "he/his" on other days to balance them, and you are > seeing the artifact of that. > > Some people might feel that we would be better off using "they" all > the time, but IMO it's such a minor thing that once it _is_ in the > tree, it's not really worth the patch noise to go and fix it up. > > >> >> Signed-off-by: Stefan Beller >> --- >> date.c | 2 +- >> 1 file changed, 1 insertion(+), 1 deletion(-) >> >> diff --git a/date.c b/date.c >> index 59dfe57..19fb439 100644 >> --- a/date.c >> +++ b/date.c >> @@ -732,7 +732,7 @@ int parse_expiry_date(const char *date, unsigned long *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 they have done in >> * the past, and by definition reflogs are the record >> * of the past, and there is nothing from the future >> * to be kept. > ```