All of lore.kernel.org
 help / color / mirror / Atom feed
From: Junio C Hamano <gitster@pobox.com>
To: Peter Colberg <peter@colberg.org>
Cc: git@vger.kernel.org
Subject: Re: [PATCH v2] Fix spelling of "occurred"
Date: Fri, 10 Jun 2016 14:15:58 -0700	[thread overview]
Message-ID: <xmqqd1noeodt.fsf@gitster.mtv.corp.google.com> (raw)
In-Reply-To: <1465585526-32187-1-git-send-email-peter@colberg.org> (Peter Colberg's message of "Fri, 10 Jun 2016 15:05:26 -0400")

Peter Colberg <peter@colberg.org> writes:

> Signed-off-by: Peter Colberg <peter@colberg.org>
> ---
>  config.c | 2 +-
>  refs.h   | 2 +-
>  2 files changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/config.c b/config.c
> index f51c56b..d7ce34b 100644
> --- a/config.c
> +++ b/config.c
> @@ -1281,7 +1281,7 @@ static void git_config_raw(config_fn_t fn, void *data)
>  		 * something went really wrong and we should stop
>  		 * immediately.
>  		 */
> -		die(_("unknown error occured while reading the configuration files"));
> +		die(_("unknown error occurred while reading the configuration files"));
>  }

For this one, I am tempted to say that it may be better to remove
the verb altogether, which would lead to a more concise error
message.

In any case, we need to postpone this until the next cycle, as it is
too late to change message strings now, as it would leave no time
for i18n/l10n team to adjust their work.

The other patch to the comment could be taken now if we wanted to,
though.

Thanks.

>  static void configset_iter(struct config_set *cs, config_fn_t fn, void *data)
> diff --git a/refs.h b/refs.h
> index 9230d47..56089d5 100644
> --- a/refs.h
> +++ b/refs.h
> @@ -345,7 +345,7 @@ struct ref_transaction *ref_transaction_begin(struct strbuf *err);
>   *     msg -- a message describing the change (for the reflog).
>   *
>   *     err -- a strbuf for receiving a description of any error that
> - *         might have occured.
> + *         might have occurred.
>   *
>   * The functions make internal copies of refname and msg, so the
>   * caller retains ownership of these parameters.

  reply	other threads:[~2016-06-10 21:16 UTC|newest]

Thread overview: 6+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2016-06-10 14:22 [PATCH] Fix spelling of "occurred" Peter Colberg
2016-06-10 18:51 ` Pranit Bauva
2016-06-10 19:05   ` [PATCH v2] " Peter Colberg
2016-06-10 21:15     ` Junio C Hamano [this message]
2016-06-10 22:04       ` Peter Colberg
2016-06-10 22:22         ` Junio C Hamano

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=xmqqd1noeodt.fsf@gitster.mtv.corp.google.com \
    --to=gitster@pobox.com \
    --cc=git@vger.kernel.org \
    --cc=peter@colberg.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.