[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