Mailing-List: contact dev-help@httpd.apache.org; run by ezmlm
Precedence: bulk
Reply-To: dev@httpd.apache.org
Date: Sat, 15 Jun 2002 15:38:27 -0700
From: Aaron Bannert <aaron@clove.org>
To: dev@httpd.apache.org
Subject: Re: cvs commit: httpd-2.0/modules/mappers mod_userdir.c
Message-ID: <20020615153827.W21255@clove.org>
Mail-Followup-To: Aaron Bannert <aaron@clove.org>, dev@httpd.apache.org
References: <20020615072059.46928.qmail@icarus.apache.org>
Mime-Version: 1.0
Content-Type: text/plain; charset=us-ascii
Content-Disposition: inline
In-Reply-To: <20020615072059.46928.qmail@icarus.apache.org>
User-Agent: Mutt/1.3.23i

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?

-aaron