[Catalyst-dev] Re: visit() and go() doc patch

Aristotle Pagaltzis pagaltzis at gmx.de
Fri Dec 30 00:17:55 GMT 2011


* Emanuele Zeppieri <emazep at gmail.com> [2011-12-29 06:15]:
> +=head2 $c->visit( $action [, \@arguments ] )
> +
>  =head2 $c->visit( $action [, \@captures, \@arguments ] )
>
> +=head2 $c->visit( $class, $method, [, \@arguments ] )
> +
>  =head2 $c->visit( $class, $method, [, \@captures, \@arguments ] )

If you are going to expand one case, why not expand the no-ref case
also, and drop the confusing/misleading [ ] metasyntax entirely?

+=head2 $c->visit( $action )
+
+=head2 $c->visit( $action, \@arguments )
+
 =head2 $c->visit( $action, \@captures, \@arguments )

+=head2 $c->visit( $class, $method )
+
+=head2 $c->visit( $class, $method, \@arguments )
+
 =head2 $c->visit( $class, $method, \@captures, \@arguments )

Just a thought. It does get a bit long that way, so I’m not sure which
evil I prefer.

Regards,
-- 
Aristotle Pagaltzis // <http://plasmasturm.org/>



More information about the Catalyst-dev mailing list