Michael Graham wrote:

> This looks great!  I like the lower case params.

that seems to be the general consensus, so that's probably the way it'll go.

>>I'm also thinking about adding the rule that
>>  module-name
>>would become
>>  ModuleName
> 
> 
> I like this a lot, too.

I think it covers the last real popular case for module names.

>>  # in httpd.conf
>>  CGIAPP_DISPATCH_TABLE_CLASS MyProject::Dispatch
>>
>>  # in MyProject/Dispatch.pm
>>  package MyProject::Dispatch;
>>  our @TABLE = (...);
> 
> 
> Would it be a bad thing to force mod_perl users to use a subclass of
> CA::Dispatch?  Something like this:
> 
>    # in httpd.conf
>    PerlHandler MyProject::Dispatch
> 
>    # in MyProject/Dispatch.pm
> 
>    package MyProject::Dispatch;
>    use base 'CGI::Application::Dispatch';
> 
>    sub dispatch_table {  ...  };

Yeah, that's probably a better idea. But I think I'll name the method
default_dispatch_table() and it will be used if no table is specified. This
makes the implementation of the default table
 [
   ''         => $default,
   ':app/:rm' => {}
 ]

pretty easy to do.

-- 
Michael Peters
Developer
Plus Three, LP


---------------------------------------------------------------------
Web Archive:  http://www.mail-archive.com/[email protected]/
              http://marc.theaimsgroup.com/?l=cgiapp&r=1&w=2
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]

Reply via email to