httpd-test-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Sergey Ten" <>
Subject RE: need a custom verify_resp function
Date Thu, 12 May 2005 18:37:28 GMT
Thank you for your help, Jacek.

As it was suggested, I replaced 'waitfor' with 'acceptcode'. Regarding
removing "global waitfor (flood_round_robin.c)". I think the diff caused
some confusion: that waitfor was a field in the url_t type, and it can't be
removed. I hope the new diff I am attaching (prepared using cvs diff -u)
will be more helpful.

Best regards,
Sergey Ten

> -----Original Message-----
> From: Jacek Prucia []
> Sent: Wednesday, May 11, 2005 3:22 PM
> To:
> Subject: Re: need a custom verify_resp function
> > Hello all,
> >
> > I have a change which implements a generic function comparing returned
> code
> > against specified as an attribute of the url. The diff is enclosed. Is
> there
> > a document how to check-in into the flood source tree?
> Yup. It is actually for httpd-dev folks, but most rules apply anyway:
> Before you prepare unified diff (see document above), I'd suggest
> removing global waitfor (flood_round_robin.c). It doesn't seem to be
> used anywhere else. Also please consider using 'acceptcode' in favor of
> 'waitfor'. Technically speaking, flood doesn't wait for anything. It
> just chceks return code, and moves forward. Besides that looks OK. If
> nobody has any objections, I'll commit it as soon as I get my
> development machine ready.
> regards,
> --
> Jacek Prucia

View raw message