On Tue, Sep 27, 2011 at 11:19 AM, C. Michael Pilato <cmpil...@collab.net> wrote:
> I've reviewed this change and voted +1 on it, approving it.

Thanks for taking a look Mike.

> However, I'd appreciate it, Paul, if you'd follow up with an "obvious fix"
> type of change (that doesn't require a vote for backport) to repair some
> sformatting problems introduced by the patch, such as this one:

Done in r1176447

Paul

> {{{
> Index: subversion/libsvn_ra_neon/ra_neon.h
> ===================================================================
> --- subversion/libsvn_ra_neon/ra_neon.h (revision 1176416)
> +++ subversion/libsvn_ra_neon/ra_neon.h (working copy)
> @@ -1,4 +1,4 @@
> -/*
> +    /*
>  * ra_neon.h : Private declarations for the Neon-based DAV RA module.
>  *
>  * ====================================================================
> }}}
>
> And the indentation changes caused by removing the "2" at the ends of
> function prototypes such as svn_fs_get_mergeinfo2() and 
> svn_ra_get_mergeinfo2().
>
> --
> C. Michael Pilato <cmpil...@collab.net>
> CollabNet   <>   www.collab.net   <>   Distributed Development On Demand
>
>

Reply via email to