Re: cvs: pear /Event_Dispatcher Dispatcher.php Notification.php package.xml /Event_Dispatcher/examples bubbling.php cancel.php
notification-class.php object.php /Event_Dispatcher/tests Console_TestListener.php Dispatcher_testcase.php test.php

From: Date: Sat, 05 Feb 2005 15:22:39 +0000
Subject: Re: cvs: pear /Event_Dispatcher Dispatcher.php Notification.php package.xml /Event_Dispatcher/examples bubbling.php cancel.php
notification-class.php object.php /Event_Dispatcher/tests Console_TestListener.php Dispatcher_testcase.php test.php
References: 1  Groups: php.pear.cvs 
Request: Send a blank email to pear-cvs+get-29153@lists.php.net to get a copy of this message
Bertrand Mansion wrote:
Also, plesae wrap the lines (both docblocks and code itself) when things approach 80 chars.
My text editor is not limited to 80 chars. As long as my code is readable and respect Coding standards, it should be ok. It is easy today to get a text editor with this "feature".
Well I think its less about editors than readability. I tend to try to not reach the 80 char limit mainly so enable the reader to have every line be meaningful on its own. Usually when I get beyond the 80 chars I have either: - too much code in a method - I have failed to bail out earlier using a return instead of nesting if statements deeply - or the code will gain readability by aliasing a deep array or object accessing statement (this should also not really affect performance in any meaningful way) Therefore I am with Dan on this one. regards, Lukas

« previous php.pear.cvs (#29153) next »