Re: [PATCH] svn_error_t * return value

Karl Fogel <kfogel@galois.ch.collab.net> writes:
> Yoshiki Hayashi <yoshiki@xemacs.org> writes:
> > Some more lightweight stuff. While I was reading, I noticed
> > some functions use just 0 instead of SVN_NO_ERROR as their
> > return values.
>
> Thanks for noticing! I think this might have been a conscious
> stylistic decision on JimB's part; will leave this patch for him to
> apply or not.

(translation: "We've found that when one steps on one of Jim's little
style issues one gets one's head bitten off, so we don't mess with
his stuff much.")

It was a conscious choice, but I can't justify it any more (I use '\0'
instead of 0, for example), so I'm going to apply the patch.
Received on Sat Oct 21 14:36:22 2006