Stas Bekman wrote:
Philippe M. Chiasson wrote:
For nothing, really. I must apologize for the confusion. I started trying to get ->limited to work, and for that, I needed to open up Apache::MethodList.
:)
I abandonned that effort too late for a simpler implementation of ap_method_is_limited().
Attached are the 2 patches, separated. (The second one possibly usefull in the future if/when we decide we need Apache::MethodList)
+1 for the first one.
On it's way.
As for Apache::MethodList I don't want your efforts to go wasted, but on the other hand I can't see where it may come in handy. And if you commit it, you need to write a manpage for it :) So it's up to you, if you feel like writing a manpage for it, go with it, if not just add a note somewhere in todo/features_maybe and add a url of the patch from marc.'s mail archive.
http://marc.theaimsgroup.com/?l=apache-modperl-dev
I did write the tests for it, but I agree with you. There are only 2-3 fields I saw that were method_lists and I can't think of a reason why it would be usefull for now. So in the archives it goes.
-- -------------------------------------------------------------------------------- Philippe M. Chiasson m/gozer\@(apache|cpan|ectoplasm)\.org/ GPG KeyID : 88C3A5A5 http://gozer.ectoplasm.org/ F9BF E0C2 480E 7680 1AE5 3631 CB32 A107 88C3A5A5
signature.asc
Description: OpenPGP digital signature
