On Aug 31, 2013, at 2:00 PM, David Deutsch <skoremail@gmail.com> wrote:Cleanup: https://github.com/daviddeutsch/roundcubemail/blob/37167c5ce1c00cb4f42b7f59a9ff56b81b3cd874/plugins/managesieve/lib/Roundcube/rcube_sieve_engine.php#L219Another possibly neat comparison:Original: https://github.com/roundcube/roundcubemail/blob/master/plugins/managesieve/lib/Roundcube/rcube_sieve_engine.php#L206
The code may be /slightly/ less DRY, but, I find, a lot more readable.Definitely more readable.I must also say that so far, I find very few yoda conditions! ;-)Find them you will.Cor
_______________________________________________
Roundcube Development discussion mailing list
dev@lists.roundcube.net
http://lists.roundcube.net/mailman/listinfo/dev