I personally find 120 characters to be the best balance. On the bright side, I expect that once we can use the diamond operator in JDK 7, the pressure for longer lines will decrease.

-- Blake Sullivan

On 10/28/11 10:33 AM, Gerhard Petracek wrote:
@80: -1!
@rest: +0

regards,
gerhard

http://www.irian.at

Your JSF powerhouse -
JSF Consulting, Development and
Courses in English and German

Professional Support for Apache MyFaces



2011/10/28 Volker Weber <v.we...@inexso.de <mailto:v.we...@inexso.de>>

    Hi Mark,

    2011/10/28 Mark Struberg <strub...@yahoo.de
    <mailto:strub...@yahoo.de>>:
    > Volker, source code is no newspaper.

    just wanted to support the statement of easier reading in smaller
    columns, of cause code is no newspaper, but it still need easy
    reading.

    I don't like to scroll left and right to read the code, and even at
    work were i got the widest screen the 1920px did not suffice to see
    more than 120 characters with the project and structure sidebars left
    and right, which i would not like to miss.


    Regards,
       Volker


    >
    > Imo 80 chars is definitely fine for C or perl with cryptic
    syntax (programmed that myself for 20 years) but it's not nice for
    languages where descriptive variable and method names are
    'socially accepted' ;)
    >
    >
    > LieGrue,
    > strub
    >
    >
    >
    > ----- Original Message -----
    >> From: Volker Weber <v.we...@inexso.de <mailto:v.we...@inexso.de>>
    >> To: MyFaces Development <dev@myfaces.apache.org
    <mailto:dev@myfaces.apache.org>>; Mark Struberg <strub...@yahoo.de
    <mailto:strub...@yahoo.de>>
    >> Cc:
    >> Sent: Friday, October 28, 2011 9:22 AM
    >> Subject: Re: [VOTE] extend maximum allowed line length from 120
    to 160
    >>
    >> Hi,
    >>
    >> -1.
    >>
    >> In my opinion 160 characters is much to wide, the current 120
    is not
    >> the preferred, but the allowed max width.
    >> I vote for 80 characters as preferred max width.
    >>
    >> In general reading is easier if the text is not too wide, thats why
    >> newspaper articles are layouted in columns.
    >>
    >>
    >> Regards,
    >>     Volker
    >>
    >> 2011/10/26 Mark Struberg <strub...@yahoo.de
    <mailto:strub...@yahoo.de>>:
    >>>  Hi!
    >>>
    >>>  Currently we have really long and very descriptive variable
    names in
    >> MyFaces.
    >>>
    >>>  I personally like that, but due to that we are really often
    exceeding the
    >> 120 character per line.
    >>>
    >>>  Thus my question: should we extend this from 120 to 160
    characters being
    >> allowed per line?
    >>>
    >>>  [+1] yup make 160 the max default
    >>>  [0] don't care
    >>>  [-1] nope, let's stick with 120
    >>>
    >>>  open for 72h ...
    >>>
    >>>
    >>>  Please make use of your vote, because I will activate the
    checkstyle checks
    >> soon ;)
    >>>
    >>>  here is my +1.
    >>>
    >>>  LieGrue,
    >>>  strub
    >>>
    >>>
    >>
    >>
    >>
    >> --
    >> inexso - information exchange solutions GmbH
    >> Ofener Str. 30      | 26121 Oldenburg
    >> Tel.: +49 441 219 730 56 <tel:%2B49%20441%20219%20730%2056> |
    >> FAX: +49 441 219 730 66 <tel:%2B49%20441%20219%20730%2066> |
    eMail: volker.we...@inexso.de <mailto:volker.we...@inexso.de>
    >>
    >> Firmensitz: Oldenburg | Amtsgericht Oldenburg HRB 205251
    >> Geschäftsführer: Stefan Schulte, Michael Terschüren
    >>
    >



    --
    inexso - information exchange solutions GmbH
    Ofener Str. 30      | 26121 Oldenburg
    Tel.: +49 441 219 730 56 <tel:%2B49%20441%20219%20730%2056> |
    FAX: +49 441 219 730 66 <tel:%2B49%20441%20219%20730%2066> |
    eMail: volker.we...@inexso.de <mailto:volker.we...@inexso.de>

    Firmensitz: Oldenburg | Amtsgericht Oldenburg HRB 205251
    Geschäftsführer: Stefan Schulte, Michael Terschüren



Reply via email to