On lun., 2013-11-25 at 23:18 -0800, javierag wrote:
> > @@ -4473,6 +4489,22 @@ sub checkAllProperty
> >      return $self->{'table'}->{'checkAll'};
> >  }
> >  
> > +sub checkAllControls
> 
> I think is very misguided, it arbitrary decides that a map should be a
> one-liner (and why not anonymous-functions by example). Just check
> this very case it just concatenate two strings and if I convert it to
> a line it becames unreadable.

I think you didn't read it carefully, or you should misinterpreted.

It states clearly "Mappings with multiple statements are even worse",
not one-liner but more than one statement on a map/grep.

Here, I'd suggest to do it in a for-loop or put it in a function the
action as he suggested, but three lines for a map seems overkill for me.

Anyway, just a discussion on how to do things.

Best,