{"thread":{"id":"38001","subject":"[PATCH] date.c: phrase a comment gender neutral","startedAt":"2014-11-18T19:24:28Z","lastAt":"2014-11-18T20:07:24Z","messageCount":4,"participants":["Stefan Beller","Jonathan Nieder","Junio C Hamano"],"isPatch":true,"patchVersion":1,"patchTotal":null},"messages":[{"id":"252110","messageId":"1416338668-30683-1-git-send-email-sbeller@google.com","threadId":"38001","inReplyTo":null,"subject":"[PATCH] date.c: phrase a comment gender neutral","fromName":"Stefan Beller","fromEmail":"sbeller@google.com","sentAt":"2014-11-18T19:24:28Z","receivedAt":"2014-11-18T19:24:28Z","isPatch":true,"sender":{"key":"stefanbeller@gmail.com","avatar":"https://avatars.githubusercontent.com/u/455868?v=4"},"body":"When looking for suitable functions to print dates,\nI found this gem. Let's make it gender neutral as\nnowadays people get upset by this gender debate.\n\nSigned-off-by: Stefan Beller <sbeller@google.com>\n---\n date.c | 2 +-\n 1 file changed, 1 insertion(+), 1 deletion(-)\n\ndiff --git a/date.c b/date.c\nindex 59dfe57..19fb439 100644\n--- a/date.c\n+++ b/date.c\n@@ -732,7 +732,7 @@ int parse_expiry_date(const char *date, unsigned long *timestamp)\n \t\t/*\n \t\t * We take over \"now\" here, which usually translates\n \t\t * to the current timestamp.  This is because the user\n-\t\t * really means to expire everything she has done in\n+\t\t * really means to expire everything they have done in\n \t\t * the past, and by definition reflogs are the record\n \t\t * of the past, and there is nothing from the future\n \t\t * to be kept.\n-- \n2.2.0.rc2.5.gf7b9fb2\n"},{"id":"252114","messageId":"20141118194242.GJ6527@google.com","threadId":"38001","inReplyTo":"1416338668-30683-1-git-send-email-sbeller@google.com","subject":"Re: [PATCH] date.c: phrase a comment gender neutral","fromName":"Jonathan Nieder","fromEmail":"jrnieder@gmail.com","sentAt":"2014-11-18T19:42:42Z","receivedAt":"2014-11-18T19:42:42Z","isPatch":true,"sender":{"key":"jrnieder@gmail.com","avatar":"https://avatars.githubusercontent.com/u/281595?v=4"},"body":"Stefan Beller wrote:\n\n> When looking for suitable functions to print dates,\n> I found this gem. Let's make it gender neutral as\n> nowadays people get upset by this gender debate.\n\nEh, I'm not upset.\n\nHope that helps,\nJonathan\n"},{"id":"252116","messageId":"xmqq4mtwwa1n.fsf@gitster.dls.corp.google.com","threadId":"38001","inReplyTo":"1416338668-30683-1-git-send-email-sbeller@google.com","subject":"Re: [PATCH] date.c: phrase a comment gender neutral","fromName":"Junio C Hamano","fromEmail":"gitster@pobox.com","sentAt":"2014-11-18T20:03:48Z","receivedAt":"2014-11-18T20:03:48Z","isPatch":true,"sender":{"key":"gitster@pobox.com","avatar":"https://avatars.githubusercontent.com/u/54884?v=4"},"body":"Stefan Beller <sbeller@google.com> writes:\n\n> When looking for suitable functions to print dates,\n> I found this gem. Let's make it gender neutral as\n> nowadays people get upset by this gender debate.\n\nFor some time I used to use \"she/her\" on Mondays, Wednesdays and\nFridays and \"he/his\" on other days to balance them, and you are\nseeing the artifact of that.\n\nSome people might feel that we would be better off using \"they\" all\nthe time, but IMO it's such a minor thing that once it _is_ in the\ntree, it's not really worth the patch noise to go and fix it up.\n\n\n>\n> Signed-off-by: Stefan Beller <sbeller@google.com>\n> ---\n>  date.c | 2 +-\n>  1 file changed, 1 insertion(+), 1 deletion(-)\n>\n> diff --git a/date.c b/date.c\n> index 59dfe57..19fb439 100644\n> --- a/date.c\n> +++ b/date.c\n> @@ -732,7 +732,7 @@ int parse_expiry_date(const char *date, unsigned long *timestamp)\n>  \t\t/*\n>  \t\t * We take over \"now\" here, which usually translates\n>  \t\t * to the current timestamp.  This is because the user\n> -\t\t * really means to expire everything she has done in\n> +\t\t * really means to expire everything they have done in\n>  \t\t * the past, and by definition reflogs are the record\n>  \t\t * of the past, and there is nothing from the future\n>  \t\t * to be kept.\n"},{"id":"252117","messageId":"CAGZ79kYtV3d0iTuzuh=+pzSOYJ8O3MUS4YFq2FbwS-pinU2+iA@mail.gmail.com","threadId":"38001","inReplyTo":"xmqq4mtwwa1n.fsf@gitster.dls.corp.google.com","subject":"Re: [PATCH] date.c: phrase a comment gender neutral","fromName":"Stefan Beller","fromEmail":"sbeller@google.com","sentAt":"2014-11-18T20:07:24Z","receivedAt":"2014-11-18T20:07:24Z","isPatch":true,"sender":{"key":"stefanbeller@gmail.com","avatar":"https://avatars.githubusercontent.com/u/455868?v=4"},"body":"ok.\n\nI'll stop sending such gender related nits.\n\nOn Tue, Nov 18, 2014 at 12:03 PM, Junio C Hamano <gitster@pobox.com> wrote:\n> Stefan Beller <sbeller@google.com> writes:\n>\n>> When looking for suitable functions to print dates,\n>> I found this gem. Let's make it gender neutral as\n>> nowadays people get upset by this gender debate.\n>\n> For some time I used to use \"she/her\" on Mondays, Wednesdays and\n> Fridays and \"he/his\" on other days to balance them, and you are\n> seeing the artifact of that.\n>\n> Some people might feel that we would be better off using \"they\" all\n> the time, but IMO it's such a minor thing that once it _is_ in the\n> tree, it's not really worth the patch noise to go and fix it up.\n>\n>\n>>\n>> Signed-off-by: Stefan Beller <sbeller@google.com>\n>> ---\n>>  date.c | 2 +-\n>>  1 file changed, 1 insertion(+), 1 deletion(-)\n>>\n>> diff --git a/date.c b/date.c\n>> index 59dfe57..19fb439 100644\n>> --- a/date.c\n>> +++ b/date.c\n>> @@ -732,7 +732,7 @@ int parse_expiry_date(const char *date, unsigned long *timestamp)\n>>               /*\n>>                * We take over \"now\" here, which usually translates\n>>                * to the current timestamp.  This is because the user\n>> -              * really means to expire everything she has done in\n>> +              * really means to expire everything they have done in\n>>                * the past, and by definition reflogs are the record\n>>                * of the past, and there is nothing from the future\n>>                * to be kept.\n>\n"}]}