Mailing-List: contact new-httpd-help@apache.org; run by ezmlm
Precedence: bulk
Reply-To: new-httpd@apache.org
Date: Sun, 7 Jan 2001 13:17:55 -0800
From: Jon Travis <jtravis@covalent.net>
To: new-httpd@apache.org
Subject: Re: index.html not served w/mod_autoindex active
Message-ID: <20010107131754.A15512@covalent.net>
References: <Pine.LNX.4.21.0101070806230.13039-100000@koj>
 <3A58AC5D.C1C40F8E@algroup.co.uk> <20010107125208.A15047@covalent.net>
 <3A58DB70.4A9F07F1@algroup.co.uk>
Mime-Version: 1.0
Content-Type: text/plain; charset=us-ascii
Content-Disposition: inline
User-Agent: Mutt/1.2.5i
In-Reply-To: <3A58DB70.4A9F07F1@algroup.co.uk>;
 from ben@algroup.co.uk on Sun, Jan 07, 2001 at 09:11:12PM +0000

On Sun, Jan 07, 2001 at 09:11:12PM +0000, Ben Laurie wrote:
> Jon Travis wrote:
> > 
> > On Sun, Jan 07, 2001 at 05:50:21PM +0000, Ben Laurie wrote:
> > > rbb@covalent.net wrote:
> > > >
> > > > > > Do not consider this a veto in any way shape or form, this is an opinion,
> > > > > > that we shouldn't have to call every handler.  Please implement whatever
> > > > > > solution you like best.
> > > > >
> > > > > Since no-one else has expressed an opinion, I shall. :-)
> > > >
> > > > Cool.  I look forward to seeing it.  :-)  I am sorry that I held things up
> > > > this long.  I didn't realize that I was the bottleneck.  If I had, I would
> > > > have gotten out of the way much sooner.  :-)
> > >
> > > Don't sweat it - I didn't have time to do it any sooner, anyway!
> > 
> > I hope this is the correct thread to respond on this to, so here goes:
> > 
> > I just saw the commits which were made to hookize the handlers.  I have
> > a few comments:
> > 
> >     1 - AP_DECLARE_HOOK for the handler should probably not be in http_config.h
> >         It should probably be in http_request.h
> 
> Probably correct.
> 
> >     2 - A new hook prototype has been made.  One that takes the handlername, and
> >         the request.  This sucks.  Since the handler information is gleaned
> >         from the request anyway, why not keep the old prototype for the
> >         handler, and make the registered hook use some function to grab the
> >         handler from the request_rec?
> 
> Because then it has to be done every time, which is inefficient. Why
> does it suck?

Because it makes me have to write more code.. ;-)  mod_snake handles all
the request_rec based handlers in a same generic way.  By changing the
prototype of this function you kinda break that.. :-(  You are doing it
everytime in ap_invoke_handler anyway, so the only inneficiency is that 
introduced is an extra function call.  And if you wanted, you could add
another field to the request_rec, and just manipulate that within
your ap_invoke_handler routine, or whatever.   That would save the 
overhead of the function call.

-- Jon