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

Re: svn commit: r1849737 - in /subversion/trunk/subversion/bindings/cxx: include/svnxx.hpp include/svnxx/exception.hpp src/aprwrap/pool.hpp src/exception.cpp src/private/exception-private.hpp tests/test_exceptions.cpp

From: Branko Čibej <brane_at_apache.org>
Date: Thu, 27 Dec 2018 00:33:19 +0100

On 26.12.2018 23:41, Daniel Shahaf wrote:
> Branko Čibej wrote on Wed, 26 Dec 2018 23:28 +0100:
>> On 26.12.2018 22:41, Branko Čibej wrote:
>>> On 26.12.2018 19:50, Daniel Shahaf wrote:
>>>> brane_at_apache.org wrote on Wed, 26 Dec 2018 04:28 +0000:
>>>>> + * - apache::subversion::svnxx::canceled\n
>>>>> + * Thrown when an operation was canceled
>>>>> + * + apache::subversion::svnxx::cancel\n
>>>>> + * Thrown by user callbacks to terminate iteration
>>>> I think the last two names here are too similar; it'll be way too easy
>>>> to mis-tab-complete them for each other. Suggest to rename the second
>>>> one to stop_iteration to match the name of SVN_ERR_STOP_ITERATION.
>>> Actually users can't create objects of type 'svn::canceled' — or at
>>> least not without a lot of effort, as there are no public constructors.
>>> I would hope a smart IDE would notice, or else at least a smart user
>>> will. :)
>>>
>>> But yes, I hear you. stop_iteration is a good suggestion.
>>
>> Out of interest ... where is SVN_ERR_STOP_ITERATION defined? I can't
>> find it on trunk.
> Sorry, SVN_ERR_ITER_BREAK. StopIteration is a Python exception class.

Right. I admit I also keep thinking that "Of course we have
SVN_ERR_STOP_ITERATION." :)

-- Brane
Received on 2018-12-27 00:33:26 CET

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