httpd-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Ben Laurie <...@gonzo.ben.algroup.co.uk>
Subject Re: 04a_ExtraPath.0.8.14.patch
Date Tue, 10 Oct 1995 09:15:18 GMT
> 
> > > 	if ((r->finfo.st_mode == 0 || (r->path_info && *r->path_info))
{
> > 
> > Well, I inherited the "r->finfo.st_mode == 0", and I must say that I haven't
> > entirely decided what it means. So if it is that you doubt, feel free.
> > Otherwise I see nothing wrong.
> 
> It doesn't compile.  The () pairing is rong.

Sabotage!

Actually, this is most odd. The extra bracket (the first one) is NOT PRESENT
in the original patch (on my machine). The line reads:

if (r->finfo.st_mode == 0 || (r->path_info && *r->path_info)) {

in the original. How on Earth did that extra bracket get there? Obviously
it was not present when I generated the patch, I compiled and tested it!
And it certainly isn't there now. But it is there in the patches dir.
Did someone helpfully "correct" it?

> > Ben Laurie                  Phone: +44 (181) 994 6435
> 
> Ay.

-- 
Ben Laurie                  Phone: +44 (181) 994 6435
Freelance Consultant        Fax:   +44 (181) 994 6472
and Technical Director      Email: ben@algroup.co.uk
A.L. Digital Ltd,
London, England.

Mime
View raw message