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

[Giovanni Bajo <ra...@develer.com>]

Madan U S <ma...@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