Re: [PATCH] Fix to svn_time_to_human_cstring() - take 3

On Sun, Apr 04, 2004 at 04:28:17PM -0300, Nicol?s_Lichtmaier wrote:
> + else
> + {
> + const char *utf8_string;
> + svn_error_t *err;
>
> + err = svn_utf_cstring_to_utf8(&utf8_string, human_datestr, pool);
> + if(err)
> + {
> + *curptr = '\0';
> + svn_error_clear(err);
> + }
> + else
> + apr_cpystrn(curptr, utf8_string, SVN_TIME__MAX_LENGTH - len);
> + }
> +

You're also not following the convention of putting a space before a (.
This is flexible, but this file appears to be consistent in this case.
So you should follow what's being used in that file.

-- 
Ben Reser <ben@reser.org>
http://ben.reser.org
"Conscience is the inner voice which warns us somebody may be looking."
- H.L. Mencken
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@subversion.tigris.org
For additional commands, e-mail: dev-help@subversion.tigris.org