[svn.haxx.se] · SVN Dev · SVN Users · SVN Org · TSVN Dev · TSVN Users · Subclipse Dev · Subclipse Users · this month's index

Re: [PATCH] Combine conflict callback API error messages differing only by punctuation

From: Daniel Shahaf <danielsh_at_elego.de>
Date: Fri, 18 Jan 2013 19:28:24 +0200

+1 for the first two hunks. The third one seems to be an unnecessary
whitespace-only change.

Andreas Stieger wrote on Fri, Jan 18, 2013 at 17:25:06 +0000:
> Hello,
>
> [[[
> Combine conflict callback API error messages differing only by punctuation
>
> * subversion/libsvn_wc/conflicts.c
> (generate_propconflict,resolve_text_conflicts): Combine messages which
> differ only by punctuation, remove trailing period from another
> ]]]
>
> Andreas

> Index: subversion/libsvn_wc/conflicts.c
> ===================================================================
> --- subversion/libsvn_wc/conflicts.c (revision 1435158)
> +++ subversion/libsvn_wc/conflicts.c (working copy)
> @@ -1386,7 +1386,7 @@ generate_propconflict(svn_boolean_t *conflict_rema
> *conflict_remains = TRUE;
> return svn_error_create(SVN_ERR_WC_CONFLICT_RESOLVER_FAILURE,
> NULL, _("Conflict callback violated API:"
> - " returned no results."));
> + " returned no results"));
> }
>
>
> @@ -1428,7 +1428,7 @@ generate_propconflict(svn_boolean_t *conflict_rema
> return svn_error_create
> (SVN_ERR_WC_CONFLICT_RESOLVER_FAILURE,
> NULL, _("Conflict callback violated API:"
> - " returned no merged file."));
> + " returned no merged file"));
> else
> {
> svn_stringbuf_t *merged_stringbuf;
> @@ -1713,8 +1713,8 @@ resolve_text_conflicts(svn_skel_t **work_items,
> scratch_pool));
> if (result == NULL)
> return svn_error_create(SVN_ERR_WC_CONFLICT_RESOLVER_FAILURE, NULL,
> - _("Conflict callback violated API: "
> - "returned no results"));
> + _("Conflict callback violated API:"
> + " returned no results"));
>
> if (result->save_merged)
> {
Received on 2013-01-18 18:29:17 CET

This is an archived mail posted to the Subversion Dev mailing list.

This site is subject to the Apache Privacy Policy and the Apache Public Forum Archive Policy.