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

Re: [PATCH] Bug in svn_fs_paths_changed2() Python bindings?

From: Alexey Neyman <stilor_at_att.net>
Date: Wed, 18 Aug 2010 16:30:53 -0700

There was a race condition :) C. Michael Pilato committed the same patch at
the same time as I sent updated version, with less than 3 minutes difference.

Regards,
Alexey.

On Wednesday, August 18, 2010 03:54:43 pm Gavin Beau Baumanis wrote:
> Ping. The renewed patch submission has received no comments.
>
>
> Gavin.
>
> On 12/08/2010, at 6:12 AM, Alexey Neyman wrote:
> > On Wednesday, August 11, 2010 12:10:41 pm C. Michael Pilato wrote:
> >> On 08/10/2010 09:22 PM, Alexey Neyman wrote:
> >>> Okay, try again:
> >>>
> >>> [[[
> >>> Fix the type of structures returned in bindings from
> >>> svn_fs_paths_changed2().
> >>>
> >>> * subversion/include/svn_fs.h
> >>>
> >>> (svn_fs_paths_changed2): Rename the argument from changed_paths_p to
> >>> changed_paths_p2, so that it's different from argument to
> >>> svn_fs_paths_changed().
> >>
> >> Minor nit -- I think 'changed_paths2_p' would be a better name. To me,
> >> a number at the very end of a parameter name says "differentiator" (as
> >> in 'strcmp(str1, str2)') whereas having that '2' closer to the
> >> 'changed_paths' name says "version iterator". But like I said, it's a
> >> minor nit.
> >>
> >> Otherwise, +1 on the patch.
> >
> > Agreed. Updated patch attached.
> >
> > Regards,
> > Alexey.
> > <fs_paths_changed2.diff>
Received on 2010-08-19 01:32:02 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.