Applied in rev 5162. Thanks!
Cheers,
-g
On Fri, Feb 28, 2003 at 05:00:21PM +0100, Marcus Comstedt wrote:
>
> Greg Hudson <ghudson@MIT.EDU> writes:
>
> > We should never ever add null pointer handling to a function without
> > explicitly noting in its doc string that it handles null pointers.
> >
> > And we shouldn't do it at all unless it makes things significantly more
> > convenient, which in this case it doesn't. So I prefer the session.c
> > patch.
>
> Ok. Here it is again, with all TABs replaced with spaces to make Mike
> happy, courtesy of M-x untabify...
>
> Index: subversion/libsvn_ra_dav/session.c
> ===================================================================
> --- subversion/libsvn_ra_dav/session.c (revision 5152)
> +++ subversion/libsvn_ra_dav/session.c (working copy)
> @@ -176,8 +176,12 @@
> SVN_CONFIG_OPTION_NEON_DEBUG_MASK, NULL);
> }
>
> - server_group = svn_config_find_group(cfg, requested_host,
> - SVN_CONFIG_SECTION_GROUPS,
> pool);
> + if (cfg)
> + server_group = svn_config_find_group(cfg, requested_host,
> + SVN_CONFIG_SECTION_GROUPS,
> pool);
> + else
> + server_group = NULL;
> +
> if (server_group)
> {
> svn_config_get(cfg, proxy_host, server_group,
>
>
> // Marcus
>
>
>
> ---------------------------------------------------------------------
> To unsubscribe, e-mail: dev-unsubscribe@subversion.tigris.org
> For additional commands, e-mail: dev-help@subversion.tigris.org
--
Greg Stein, http://www.lyra.org/
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@subversion.tigris.org
For additional commands, e-mail: dev-help@subversion.tigris.org
Received on Sat Mar 1 16:09:27 2003