I can understand the positions of Kevin and Claudio;
they both have a point; one has to be careful with the postfix "if".
I would not go as far and prohibit the postfix "if" - sometimes it
allows nice. readable formulations.
Best regards,
Zeno
On Mon, Jul 18, 2011 at 9:25 AM, Sebastian Willing
<sebastian.will...@web.de> wrote:
> 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
>
--
MyMediaLite Recommender System Library: http://ismll.de/mymedialite
_______________________________________________
Padre-dev mailing list
Padre-dev@perlide.org
http://mail.perlide.org/mailman/listinfo/padre-dev
