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 <[email protected]> wrote: > I'ld go with Zeno: Advoid "and", gut don't care which if US being used. > > Sebastian > > Gabor Szabo <[email protected]> schrieb: > >>On Sun, Jul 17, 2011 at 12:57 PM, Gabor Szabo <[email protected]> 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 >>[email protected] >>http://mail.perlide.org/mailman/listinfo/padre-dev > _______________________________________________ > Padre-dev mailing list > [email protected] > http://mail.perlide.org/mailman/listinfo/padre-dev > -- MyMediaLite Recommender System Library: http://ismll.de/mymedialite _______________________________________________ Padre-dev mailing list [email protected] http://mail.perlide.org/mailman/listinfo/padre-dev
