Mailing List Archive

On 29/12/2011, at 4:11 PM, Emanuele Zeppieri wrote:

> Though formally correct, the visit() and go() documented signatures can be misleading, since they say (e.g. for visit):
>
> $c->visit( $action [, \@captures, \@arguments ] )
> ...
>
> while visit() can be called also with a single arrayref argument, with the following /unexpected/ semantics:
>
> $c->visit( $action, \@arguments );
>
> This is a small doc patch to clarify that (well, unless there is some reason to keep it undocumented - but this way the user can easily be misled).
>
> It should also close the #59601 RT ticket:
> https://rt.cpan.org/Public/Bug/Display.html?id=59601


Thanks, applied in git commit hash: 8431b9cc52d1
_______________________________________________
Catalyst-dev mailing list
Catalyst-dev@lists.scsys.co.uk
http://lists.scsys.co.uk/cgi-bin/mailman/listinfo/catalyst-dev

* Emanuele Zeppieri <emazep@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/>

_______________________________________________
Catalyst-dev mailing list
Catalyst-dev@lists.scsys.co.uk
http://lists.scsys.co.uk/cgi-bin/mailman/listinfo/catalyst-dev