On Sun, Jul 17, 2011 at 12:57 PM, 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
>
I have united all 4 calls by moving the condition inside the
subroutine. So this case is not relevant any more but the general
question is still there.
regards
Gabor
_______________________________________________
Padre-dev mailing list
Padre-dev@perlide.org
http://mail.perlide.org/mailman/listinfo/padre-dev
