+1 for having 120 length as a recommendation . Enforcement would compromise on readability.
On Wed, Dec 2, 2015 at 11:56 AM, Chandni Singh <[email protected]> wrote: > I agree with Thomas. Having 120 length should be a recommendation not an > enforcement. > > Chandni > > > > On Tue, Dec 1, 2015 at 10:13 PM, Priyanka Gugale <[email protected] > > > wrote: > > > +1 to have fixed length. > > We need to check if there any way to help editors split the big strings > in > > better way. > > Or for such exceptions, coder can format the part manually, adhering to > the > > length restriction and giving better readability. > > > > -Priyanka > > > > On Wed, Dec 2, 2015 at 11:36 AM, Pradeep A. Dalvi < > [email protected] > > > > > wrote: > > > > > -1 for 120 hard stop. There should be guideline to follow 120 line > > length, > > > not an enforcement. > > > Few characters ahead of 120 limit shall be allowed, if otherwise > > > compromises readability. > > > > > > On Wed, Dec 2, 2015 at 11:20 AM, Timothy Farkas <[email protected]> > > > wrote: > > > > > > > +1 to keep 120 line length. In my opinion it improves readability > > because > > > > it allows you to read code by only scrolling up and down. If you mix > > > having > > > > to scroll up, down, left, and right into the mix it can become > > difficult > > > to > > > > read code. > > > > > > > > On Tue, Dec 1, 2015 at 9:38 PM, Vlad Rozov <[email protected]> > > > > wrote: > > > > > > > > > +1 to keep 120 hard stop enforcement and not to rely on IDE > > formatting > > > to > > > > > find wrapping point. I agree that wrapping string literal not > always > > > help > > > > > with readability, but overall I think that enforcing a hard stop on > > the > > > > > line length help with writing better code especially after going > > > through > > > > an > > > > > exercise of fixing all code style violations in the buffer server. > > > > > Additionally some string literals may span multiple lines and will > > > > require > > > > > breaking anyway. > > > > > > > > > > Thank you, > > > > > > > > > > Vlad > > > > > > > > > > > > > > > On 12/1/15 21:19, Thomas Weise wrote: > > > > > > > > > >> A while ago, we discussed max length for line length enforcement > and > > > > >> majority wanted to stop at 120 characters. > > > > >> > > > > >> Since then Vlad has fixed code style violations for one of the > > > modules: > > > > >> > > > > >> https://github.com/apache/incubator-apex-core/pull/175 > > > > >> > > > > >> Before we continue I would like to put the line length enforcement > > > back > > > > >> for > > > > >> poll. > > > > >> > > > > >> I think it leads to undesirable results, such as breaking string > > > > literals. > > > > >> There are also instances of questionable readability gains and the > > > > breaks > > > > >> still have to be manually handled due to unwelcome IDE > auto-format. > > > > >> > > > > >> My preference would be not not enforce a line length. > > > > >> > > > > >> Opinions please. > > > > >> > > > > >> > > > > > > > > > > > > > > >
