Re: new package syntax, where do I return true?

2011-07-02 Thread Chas. Owens
On Sat, Jul 2, 2011 at 00:00, Caleb Cushing wrote: > Having a little debate over this on a Perl::Critic ticket. I'm not > sure anyone is sure and the docs don't say. > > https://rt.cpan.org/Ticket/Display.html?id=69234 > > > is it package foo { ... 1; }  or package foo { ... }; 1; ? or what? > and

Re: new package syntax, where do I return true?

2011-07-02 Thread Chas. Owens
On Sat, Jul 2, 2011 at 08:53, Chas. Owens wrote: > On Sat, Jul 2, 2011 at 00:00, Caleb Cushing wrote: >> Having a little debate over this on a Perl::Critic ticket. I'm not >> sure anyone is sure and the docs don't say. >> >> https://rt.cpan.org/Ticket/Display.html?id=69234 snip I have just rerea

Re: new package syntax, where do I return true?

2011-07-02 Thread brian d foy
[[ This message was both posted and mailed: see the "To," "Cc," and "Newsgroups" headers for details. ]] In article , Caleb Cushing wrote: > Having a little debate over this on a Perl::Critic ticket. I'm not > sure anyone is sure and the docs don't say. > > https://rt.cpan.org/Ticket/Display

Re: new package syntax, where do I return true?

2011-07-02 Thread Caleb Cushing
On Sat, Jul 2, 2011 at 1:13 AM, Flavio Poletti wrote: > i.e. it advices you to return a true value from the *file*, not from the > package. This seems to be confirmed by require's documentation: > >    The file must return true as the last statement to indicate >    success