[RCD] Codebase cleanup, PSR-1/2 compliance - and before you say no, I'm offering to do the work
skoremail at gmail.com
Sat Aug 31 20:00:01 CEST 2013
Another possibly neat comparison:
The code may be /slightly/ less DRY, but, I find, a lot more readable.
I must also say that so far, I find very few yoda conditions! ;-)
On Sat, Aug 31, 2013 at 12:54 PM, David Deutsch <skoremail at gmail.com> wrote:
> Hi Alec,
> Alright, will contain myself ;-)
> As for my comments: It's not really so much about what I like or dislike -
> I was just trying to discuss the matter. Feel free to follow up ;-)
> On Sat, Aug 31, 2013 at 7:37 AM, A.L.E.C <alec at alec.pl> wrote:
>> On 08/31/2013 02:17 AM, David Deutsch wrote:
>> > This is the first time that I'm considering breaking something up into
>> > separate methods. There's a 600 line if/elseif construct in there. I
>> > come /on/.
>> Yes. This code is quite big because of the nature of filters. We can
>> split it to separate methods, but please, do not do this now. Focus on
>> code style and simple fixes.
>> Also, I see you didn't like some of my comments on your code style. I
>> propose to wait for Thomas. I think he might have taste similar to mine.
>> Aleksander 'A.L.E.C' Machniak
>> LAN Management System Developer [http://lms.org.pl]
>> Roundcube Webmail Developer [http://roundcube.net]
>> PGP: 19359DC1 @@ GG: 2275252 @@ WWW: http://alec.pl
>> Roundcube Development discussion mailing list
>> dev at lists.roundcube.net
-------------- next part --------------
An HTML attachment was scrubbed...
More information about the dev