[dev] line wrapping

Michael Rubinsky mrubinsk at horde.org
Mon Jun 16 17:11:43 UTC 2008


Quoting "Jan Schneider" <jan at horde.org>:

> Zitat von Chuck Hagenbuch <chuck at horde.org>:
>
>> I'd like to propose letting line-wrapping changes like this:
>>
>> -            return Horde::callHook('_whups_hook_get_attributes',
>> $type, 'whups');
>> +            return Horde::callHook('_whups_hook_get_attributes', $type,
>> +                                   'whups');
>>
>> go. I don't know how many horde developers or people who read horde  
>> code have fixed 80-column screens now, but to move, moving 9  
>> characters to a new line does not increase overall code  
>> readability. Thoughts?
>
> I do, though admitted only to comply with the CS. I still think it  
> *does* increase readability, especially when the code otherwise  
> wraps on the screen like in the quoted code above.
> But I'm fine with a more relaxed CS, like an optimal line length of  
> 80 chars and a maximum line length of 90, or 100.
>

I agree with Jan. I find code like that easier to read, maybe because  
I've grown  used to it.  I'm all for a more relaxed line  
length/wrapping standard though.




Thanks,
mike

--
The Horde Project (www.horde.org)
mrubinsk at horde.org

"Time just hates me. That's why it made me an adult." - Josh Joplin
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 189 bytes
Desc: PGP Digital Signature
Url : http://lists.horde.org/archives/dev/attachments/20080616/e3501dfd/attachment.bin 


More information about the dev mailing list