i'm not completely following the new syntax that you are proposing, so please forgive if I am wide of the mark here ... but ...
IMO the existing syntax is really not *that* bad - the only mild crit I have is that Args() really means "I'm an endpoint", which it doesn't say. For me, a simple improvement would be to allow an alternative name such as "EndpointArgs" that is a bit more obvious, and keep the rest just as it is. I worry a bit that two syntaxes, achieving the same thing but looking quite different to each other, and both coexisting in docs, mailing lists, and code, are just going to add more potential confusion than they clear up. Assuming that is what would happen - that's what I gather from Zbigniew's first description. D On 12/21/07, Zbigniew Lukasiak <[EMAIL PROTECTED]> wrote: > Or maybe a better idea would be to use IsParent (or Parent) for those > actions that can have children (logical isn't it?). Then we could use > Args all the time instead of CaptureArgs (with EndPoint). > > Z. > > On Dec 20, 2007 12:31 PM, Matt S Trout <[EMAIL PROTECTED]> wrote: > > > > On Wed, Dec 19, 2007 at 02:18:17PM +0000, Zbigniew Lukasiak wrote: > > > On Dec 18, 2007 2:00 AM, Matt S Trout <[EMAIL PROTECTED]> wrote: > > > > On Mon, Dec 17, 2007 at 08:39:29PM +0000, Zbigniew Lukasiak wrote: > > > > > Yeah - some time ago I proposed to add an EndPoint attribute and get > > > > > rid of the CaptureArgs one that is not very intuitive (and use Args in > > > > > both cases). > > > > > > > > I don't remember seeing the code - if you update your patch to work > > > > against > > > > 5.80 trunk we can have a look ... > > > > > > A proof of concept implementation is in > > > Catalyst::Controller::PathPart. It replaces the attributes at parsing > > > time, I guess this would not be appropriate for the core of Catalyst. > > > I am now going to port that change into > > > lib/Catalyst/DispatchType/Chained.pm. I am not sure I'll be able to > > > do it though. > > > > Remember you can't break backcompat in the process either ... > > > > > > -- > > Matt S Trout Need help with your Catalyst or DBIx::Class > > project? > > Technical Director > > http://www.shadowcat.co.uk/catalyst/ > > Shadowcat Systems Ltd. Want a managed development or deployment platform? > > http://chainsawblues.vox.com/ http://www.shadowcat.co.uk/servers/ > > > > _______________________________________________ > > List: [email protected] > > Listinfo: http://lists.scsys.co.uk/cgi-bin/mailman/listinfo/catalyst > > Searchable archive: http://www.mail-archive.com/[EMAIL PROTECTED]/ > > Dev site: http://dev.catalyst.perl.org/ > > > > > > -- > Zbigniew Lukasiak > http://brudnopis.blogspot.com/ > > _______________________________________________ > List: [email protected] > Listinfo: http://lists.scsys.co.uk/cgi-bin/mailman/listinfo/catalyst > Searchable archive: http://www.mail-archive.com/[EMAIL PROTECTED]/ > Dev site: http://dev.catalyst.perl.org/ > -- Daniel McBrearty email : danielmcbrearty at gmail.com http://www.engoi.com http://danmcb.vox.com http://danmcb.blogger.com find me on linkedin and facebook BTW : 0873928131 _______________________________________________ List: [email protected] Listinfo: http://lists.scsys.co.uk/cgi-bin/mailman/listinfo/catalyst Searchable archive: http://www.mail-archive.com/[EMAIL PROTECTED]/ Dev site: http://dev.catalyst.perl.org/
