Re: [Svnmerge] [PATCH] svnmerge.py: all error handling to be done usingthe error() function

Madan U S <madan@collab.net> wrote:

> [[[
> All error handling to be done using the error() function.
>
> * contrib/client-side/svnmerge.py
> (error): Moved function to top of the file (well, almost ;)
> (global - during check for python version):
> (main):
> (check_old_prop_version): Replace code to print error message, exit(1)
> with call to error().
> ]]]

Thanks for the cleanup!

>> except KeyboardInterrupt:
>> # Avoid traceback on CTRL+C
>>- print "aborted by user"
>>- sys.exit(1)
>>+ err_str = "aborted by user"
>>+ error(err_str)

You don't need err_str here.

-- 
Giovanni Bajo
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@subversion.tigris.org
For additional commands, e-mail: dev-help@subversion.tigris.org