[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: Gavin Beau Baumanis <gavin_at_thespidernet.com>
Date: Thu, 19 Aug 2010 08:54:43 +1000

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 00:55:28 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.