I'ld go with Zeno: Advoid "and", gut don't care which if US being used.
Sebastian
Gabor Szabo <szab...@gmail.com> schrieb:
>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
_______________________________________________
Padre-dev mailing list
Padre-dev@perlide.org
http://mail.perlide.org/mailman/listinfo/padre-dev
