[Omaha.pm] code change ... good? bad? ugly?

Ryan Stille ryan at cfwebtools.com
Fri Mar 27 10:46:50 PDT 2009


I like the ternary.
-Ryan

Jay Hannah wrote:
> Hmm... I find this use of the ternary operator confusing so I changed 
> it...
>
>
> BEFORE:
>
>   $align = ( $self->align eq 'justify' ) ? 'left' : $self->align;
>
> AFTER:
>
>   # We've run out of words to fill a full line
>   if ($self->align eq 'justify') {
>      $self->align('left');
>   }
>
>
> What do you think?
>
>
> http://github.com/jhannah/pdf-textblock/blob/master/lib/PDF/TextBlock.pm
>
> _______________________________________________
> Omaha-pm mailing list
> Omaha-pm at pm.org
> http://mail.pm.org/mailman/listinfo/omaha-pm



More information about the Omaha-pm mailing list