You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@subversion.apache.org by kf...@collab.net on 2005/02/01 23:08:59 UTC

Re: svn commit: r12886 - trunk/subversion/libsvn_ra_dav

Philip Martin <ph...@codematters.co.uk> writes:
> > Log:
> > Check for an unusual, but not inconceivable, error case:
> >
> > * subversion/libsvn_ra_dav/props.c
> >   (svn_ra_dav__search_for_starting_props): Error if Neon was unable to
> >     parse the URL, instead of proceeding with a null parsed_url.path.
> 
> I looked into fixing this; I didn't do it because there are other
> places where ne_uri_parse is used unchecked, and some of those return
> through merge.c:end_element and I wasn't sure how to handle the error.

Okay, thanks for letting me know.  I guess my commit was still
"progress", but it would be nice to get those other places as well...

I've filed issue #2207 about this.

-Karl

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@subversion.tigris.org
For additional commands, e-mail: dev-help@subversion.tigris.org