Re: [RDBO] Manager quiet about bad syntax

2008-02-17 Thread John Siracusa
On Feb 17, 2008 2:04 PM, Sam Tregar [EMAIL PROTECTED] wrote: Params::Validate is pretty fast, in my experience. Could I win you over with a patch and some benchmarks showing no significant performance degradation? Maybe, but as I've said I'm not really a fan of runtime parameter validation.

Re: [RDBO] Manager quiet about bad syntax

2008-02-17 Thread Sam Tregar
On Feb 17, 2008 3:36 PM, John Siracusa [EMAIL PROTECTED] wrote: Maybe, but as I've said I'm not really a fan of runtime parameter validation. I'm a horrible typist which makes me a big fan. Doing it in just one place may have a minimal impact, but then the question becomes, why isn't it

[RDBO] Manager quiet about bad syntax

2008-02-14 Thread Sam Tregar
Hey all. I'm working on adjusting to Rose from a long history with CDBI, so I do this kind of thing a lot: $foos = My::Foo::Manager-get_foos(bar = big); When I really mean: $foos = My::Foo::Manager-get_foos(query = [bar = big]); This is a nasty bug since the first incorrect call doesn't

Re: [RDBO] Manager quiet about bad syntax

2008-02-14 Thread John Siracusa
On Thu, Feb 14, 2008 at 6:14 PM, Sam Tregar [EMAIL PROTECTED] wrote: Hey all. I'm working on adjusting to Rose from a long history with CDBI, so I do this kind of thing a lot: $foos = My::Foo::Manager-get_foos(bar = big); When I really mean: $foos = My::Foo::Manager-get_foos(query =