myfaces-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Gerhard Petracek <gerhard.petra...@gmail.com>
Subject Re: [VOTE] extend maximum allowed line length from 120 to 160
Date Fri, 28 Oct 2011 17:33:15 GMT
@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.weber@inexso.de>

> Hi Mark,
>
> 2011/10/28 Mark Struberg <struberg@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.weber@inexso.de>
> >> To: MyFaces Development <dev@myfaces.apache.org>; Mark Struberg <
> struberg@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 <struberg@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 |
> >> FAX:  +49 441 219 730 66 | eMail: volker.weber@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 |
> FAX:  +49 441 219 730 66 | eMail: volker.weber@inexso.de
>
> Firmensitz: Oldenburg | Amtsgericht Oldenburg HRB 205251
> Geschäftsführer: Stefan Schulte, Michael Terschüren
>

Mime
View raw message