Hello,

I would try to avoid the "and" variant.

I am less decided on the statement modifier "if" variant vs. the
compound statement "if".

The compound statement "if" is IMHO certainly the cleanest way to
express it, on the other hand it needs 2 more lines.

Other opinions?

Best,
  Zeno

On Sun, Jul 17, 2011 at 11:57 AM, Gabor Szabo <szab...@gmail.com> wrote:
> I was just looking at Padre::Wx::Main::save_current_session and how it
> is used and found 4 use cases:
>
>
>  $self->ide->{session_autosave} and $self->save_current_session;
>
>
>  $self->save_current_session  if $self->ide->{session_autosave};
>
>  if ( $self->ide->{session_autosave} ) {
>      $self->save_current_session;
>  }
>
>  $self->ide->{session_autosave} and $self->save_current_session;
>
> So that's 3 different of saying the same.
>
> I wonder what do you think, should we start working toward
> a single style in Padre?
>
> Gabor
> _______________________________________________
> Padre-dev mailing list
> Padre-dev@perlide.org
> http://mail.perlide.org/mailman/listinfo/padre-dev
>



-- 
MyMediaLite Recommender System Library: http://ismll.de/mymedialite
_______________________________________________
Padre-dev mailing list
Padre-dev@perlide.org
http://mail.perlide.org/mailman/listinfo/padre-dev

Reply via email to