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

Re: [PATCH] Issue 1846, RA layer part, take 2

From: Greg Hudson <ghudson_at_MIT.EDU>
Date: 2004-06-19 00:20:52 CEST

+ /* Read the hash items. */
+ is_done = FALSE;
+
+ *locations = apr_hash_make(pool);
+
+ /* ### Check for error and return not implemented. */
+ while (!is_done)

Okay, first, those blank lines aren't really doing anything except
visually separating the /* Read the hash items. */ comment from the code
it's talking about. Second, the ### comment should go, if it's talking
about what I think it's talking about. If the server encounters an
error, it will write out "done" and then send an error in the command
response.

Otherwise, everything but the DAV part looks okay to me. Fortunately,
the above is just a cosmetic change, so it shouldn't require another
iteration of testing.

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@subversion.tigris.org
For additional commands, e-mail: dev-help@subversion.tigris.org
Received on Sat Jun 19 00:21:40 2004

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.