Philippe M. Chiasson wrote:
In trying to make sense of Apache::CmdParms->limited() I realized it was
mainly useless to expose, and that there was a method ap_method_is_limited()
designed for that exact purpose.


So the following patch removed the parms->limited field and replaces it
with ap_method_is_limited().

+1

For consistency, I suggest making the
Apache::MethodList class available.

For consistency with what? What do we need this class for?

Thoughts ?

Index: docs/api/Apache/CmdParms.pod
===================================================================

       # which methods are <Limit>ed ?
-      $limit = $parms->limited;
+      $limit = $parms->method_is_limited('GET');

here and below, a more suitable return var name should be used, e.g. is_limited


-  $limit = $parms->limited;
+  $limit = $parms->method_is_limited($method);

+=item ret: C<$limit> ( boolean )

-- __________________________________________________________________ Stas Bekman JAm_pH ------> Just Another mod_perl Hacker http://stason.org/ mod_perl Guide ---> http://perl.apache.org mailto:[EMAIL PROTECTED] http://use.perl.org http://apacheweek.com http://modperlbook.org http://apache.org http://ticketmaster.com

---------------------------------------------------------------------
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]



Reply via email to