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

Re: svn commit: rev 6319 - trunk/subversion/include

From: <kfogel_at_collab.net>
Date: 2003-06-23 21:34:41 CEST

cmpilato@collab.net writes:
> > Greg Hudson noticed a documentation bug:
> >
> > * subversion/include/svn_path.h: Note that paths are non-null unless
> > otherwise documented.
> > (svn_path_uri_encode, svn_path_uri_encode): Say that these return
> > null when incoming path is null.
> Honestly, I think the right solution is to stop allowing NULL input
> here. We expect better than that for the rest of the svn_path.h
> functions, so perhaps we should file a bite-sized task for disallowing
> NULL input to these functions (and fixing calling code to ensure that
> that won't happen). Thoughts?

Sure -- though that's the solution to a different bug :-).

(We don't know if it's really bite-sized or not, I guess. The
conditions in the callers could be arbitrarily complex...)

To unsubscribe, e-mail: dev-unsubscribe@subversion.tigris.org
For additional commands, e-mail: dev-help@subversion.tigris.org
Received on Mon Jun 23 22:23:10 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.