[svn.haxx.se] · SVN Dev · SVN Users · SVN Org · TSVN Dev · TSVN Users · Subclipse Dev · Subclipse Users · this month's index

Re: [PATCH] Issue #1295 (take 2, with log)

From: <cmpilato_at_collab.net>
Date: 2003-05-09 09:51:11 CEST

=?UTF-8?B?QnJhbmtvIMSMaWJlag==?= <brane@xbc.nu> writes:

> >+/* Parse the edited comitted message and store the named paths
> >+ in the hash. */
> >+static void
> >+process_comitted_list (apr_hash_t *seen,
> >+ char *commited_list,
> >+ apr_pool_t *pool)
> >
> The docstring for this function should explain exactly what kind of
> input it expects, what it'll parse and when it'll fail. I'm also a bit
> worried about its silently doing nothing if it encounters something it
> can't parse; I think that's a potential source of many surprises and
> "error" reports; we want to avoid that, so giving a bit of though to the
> failure cases would be good here.

Additionally, I think the spelling "committed" is preferred to a
mixture of "commited" and "commited". :-)

To unsubscribe, e-mail: dev-unsubscribe@subversion.tigris.org
For additional commands, e-mail: dev-help@subversion.tigris.org
Received on Fri May 9 09:56:55 2003

This is an archived mail posted to the Subversion Dev mailing list.

This site is subject to the Apache Privacy Policy and the Apache Public Forum Archive Policy.