-----BEGIN PGP SIGNED MESSAGE-----
Hash: SHA1

On Friday 22 October 2004 19:10, Stas Bekman wrote:
> > I thought about it this night and realized that I need not to traverse
> > these lists at all. At first ist was a try to guess what to call
> > ap_input_filter_remove or ap_output_filter_remove. But if the filter is
> > not in an appropriate chain these functions do simply nothing. Hence I
> > can call them both if I don't know what kind of filter it is.
>
> I wish we could somehow know what kind of filter is in question. In which
> case we won't have needed to run wasteful CPU cycles. BTW, I've reversed
> the order of removal, since I believe we will have many more output
> filters than input ones. What do you think?

Yes, I agree that there will be much more output filters but since we cannot 
not determine if ap_remove_output_filter has actually removed the filter we 
have to call ap_remove_input_filter in any case. Hence the calling order is 
all the same.

> > The attached patch contains the updated Apache__Filter.h and a test that
> > removes INCLUDES from the output chain and DEFLATE from input.
>
> Excellent. I've massaged both quite a bit to adher to the style and
> simplify where it made sense (And found a perl bug in regex qr//m on the
> way :( see my latest report to p5p)

Looks good to me

> I'll commit it as soon as Philippe releases 1.99_17 (today?)

Thanks

Torsten
-----BEGIN PGP SIGNATURE-----
Version: GnuPG v1.2.4 (GNU/Linux)

iD8DBQFBeVL0wicyCTir8T4RAiCiAKCT1gXWqPle5r08q7BSSFEhYKvr7QCdGedI
Nszf0Y4AANkitUcjzn2CeDM=
=iQl2
-----END PGP SIGNATURE-----

--
Report problems: http://perl.apache.org/bugs/
Mail list info: http://perl.apache.org/maillist/modperl.html
List etiquette: http://perl.apache.org/maillist/email-etiquette.html

Reply via email to