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

Re: svn commit: r1355692 - in /subversion/trunk/subversion: libsvn_wc/conflicts.c libsvn_wc/conflicts.h libsvn_wc/wc_db.c tests/libsvn_wc/conflict-data-test.c

From: Greg Stein <gstein_at_gmail.com>
Date: Sat, 30 Jun 2012 07:11:47 -0400

On Jun 30, 2012 7:00 AM, <rhuijben_at_apache.org> wrote:
>...
> @@ -627,6 +630,33 @@ svn_wc__conflict_read_info(svn_wc_operat
> else if (locations)
> *locations = NULL;
>
> + if (text_conflicted)
> + {
> + svn_skel_t *c_skel;
> + SVN_ERR(conflict__get_conflict(&c_skel, conflict_skel,
> + SVN_WC__CONFLICT_KIND_TEXT));
> +
> + *text_conflicted = (c_skel != NULL);
> + }
> +
> + if (prop_conflicted)
> + {
> + svn_skel_t *c_skel;
> + SVN_ERR(conflict__get_conflict(&c_skel, conflict_skel,
> + SVN_WC__CONFLICT_KIND_PROP));
> +
> + *prop_conflicted = (c_skel != NULL);
> + }
> +
> + if (tree_conflicted)
> + {
> + svn_skel_t *c_skel;
> + SVN_ERR(conflict__get_conflict(&c_skel, conflict_skel,
> + SVN_WC__CONFLICT_KIND_TREE));
> +
> + *tree_conflicted = (c_skel != NULL);
> + }

Seriously? Maybe I need a macro to ease my reviews:

const!

>...

(and to repeat my request for blank lines between declarations and code;
those are separate semantic items, and should have visible separation)

Cheers,
-g
Received on 2012-06-30 13:12:40 CEST

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.