httpd-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Ryan Bloom" <...@covalent.net>
Subject RE: cvs commit: httpd-2.0/modules/mappers mod_userdir.c
Date Sat, 15 Jun 2002 23:53:17 GMT
> From: Aaron Bannert [mailto:aaron@clove.org]
> 
> On Sat, Jun 15, 2002 at 07:20:59AM -0000, rbb@apache.org wrote:
> >   1.50      +5 -5      httpd-2.0/modules/mappers/mod_userdir.c
> >
> >   Index: mod_userdir.c
> >
===================================================================
> >   RCS file: /home/cvs/httpd-2.0/modules/mappers/mod_userdir.c,v
> >   retrieving revision 1.49
> >   retrieving revision 1.50
> >   diff -u -r1.49 -r1.50
> >   --- mod_userdir.c	31 May 2002 16:59:13 -0000	1.49
> >   +++ mod_userdir.c	15 Jun 2002 07:20:59 -0000	1.50
> >   @@ -347,12 +347,12 @@
> >                                             r->pool)) == APR_SUCCESS
> >                                                 || rv ==
> APR_INCOMPLETE))) {
> >                r->filename = apr_pstrcat(r->pool, filename, dname,
> NULL);
> >   -	    /* XXX: Does this walk us around FollowSymLink rules?
> >   +            /* XXX: Does this walk us around FollowSymLink rules?
> >                 * When statbuf contains info on r->filename we can
save
> a syscall
> >   -	     * by copying it to r->finfo
> >   -	     */
> >   -	    if (*userdirs && dname[0] == 0)
> >   -		r->finfo = statbuf;
> >   +             * by copying it to r->finfo
> >   +             */
> >   +            if (*userdirs && dname[0] == 0)
> >   +                r->finfo = statbuf;
> >
> >                /* For use in the get_suexec_identity phase */
> >                apr_table_setn(r->notes, "mod_userdir_user", w);
> 
> Is this anything more than an intentation change, and if so did you
mean
> to include it in the same commit with the shtml AddType fix?

It is just indentation, and no I didn't mean to include it, but since it
is safe, I left it there.

Ryan



Mime
View raw message