31 Aug
2013
31 Aug
'13
5:37 a.m.
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 mean 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