[phobos] Column limits again

[Cliff Hudson]

80 characters.  Not a phobos contributor, but I've been coding for 20 years
in various languages.  Long lines are hard to read - almost invariable nasty
compound expressions (which should be factored for readability) or extremely
long identifier names (nothing wrong with this, but ids should be as long as
necessary and no longer.  There is no benefit to putting so much stuff onto
a single line.

If you don't like coding style rules, tough - welcome to the real world.
 Until we write an IDE that operates on ASTs and formats code on an
individual basis, we will continue to have to do these things.

P.S. As a general rule, maintain the formatting style of the file in which
you are making edits (spacing, bracing, etc.)

- Cliff

On Sat, Apr 9, 2011 at 4:13 PM, Simen Kjaeraas <simen.kjaras at gmail.com>wrote:

> On Sun, 10 Apr 2011 01:02:30 +0200, Andrei Alexandrescu <andrei at erdani.com>
> wrote:
>
>  I must bring this back into discussion because it seems the recent traffic
>> in the matter left the impression that "anything goes".
>>
>> We have a pull request (
>> https://github.com/D-Programming-Language/phobos/pull/27) with at least
>> one line that is 195 characters long and others of comparable lengths.
>>
>> I think as far as presenting arguments the ship has sailed. Let's now
>> agree on a maximum line length. What should that be? 80? (Yes please.) 100?
>> 120? Give me your vote for a number, but please let's avoid reviewing telex
>> rolls going forward.
>>
>
> The longest line is actually 408 characters long. Ouch.
>
> I'm firmly in the 80-char camp. That said, I'm not a Phobos contributor,
> so perhaps I should keep my mouth shut (or my fists clenched, or whatever
> the equivalent is for typing).
>
> --
> Simen
>
> _______________________________________________
> phobos mailing list
> phobos at puremagic.com
> http://lists.puremagic.com/mailman/listinfo/phobos
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.puremagic.com/pipermail/phobos/attachments/20110409/309fea3b/attachment.html>