[step 04] request: 12 steps towards a high performance server
Andrew Bennetts
andrew at canonical.com
Thu Sep 14 06:51:57 BST 2006
On Wed, Sep 13, 2006 at 11:41:10AM -0500, John Arbash Meinel wrote:
> > http://people.ubuntu.com/~andrew/bzr/BzrNewError.__str__/
> > make BzrNewError.__str__'s error handling a bit more robust and bit more
> > informative.
>
> +0.95
>
> I thought there was a test that checked the result of an unprintable
> BzrNewError. So it seems that test should be updated along with this patch.
>
> I don't see it now, but I think it would be good to add one. You can do
> it as easily as just creating a child exception which has a docstring
> with parameters that don't match the variables. (Either a %d with a
> string not a number, or a missing variable, etc)
>
> I wouldn't require the test, since I think it is a beneficial change,
> but it would be nice to get one.
I've added a test. Thanks for the review.
-Andrew.
More information about the bazaar
mailing list