[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH] Automatic lyric extenders
From: |
Werner LEMBERG |
Subject: |
Re: [PATCH] Automatic lyric extenders |
Date: |
Thu, 22 Dec 2016 20:52:33 +0100 (CET) |
>> One nit: Please avoid overlong lines in the source code; it should
>> be limited to 80 characters per line to improve legibility.
>
> Well, a number of lines are longer than 80 characters, but most are
> shorter than their predecessors. If the 80 character limit should
> be obeyed, it would be no problem to reformat e.g.
> lyric-extender.cc.
Complete formatting of source code files should be avoided, since it
makes it hard to compare them with older versions of the files.
However, proper formatting of new or changed source code lines should
be preferred.
Werner
- Re: [PATCH [uploaded to Rietveld]] Automatic lyric extenders, (continued)
- Re: [PATCH [uploaded to Rietveld]] Automatic lyric extenders, Trevor Daniels, 2016/12/25
- Re: [PATCH [uploaded to Rietveld]] Automatic lyric extenders, Alexander Kobel, 2016/12/25
- Re: [PATCH [uploaded to Rietveld]] Automatic lyric extenders, Alexander Kobel, 2016/12/25
- Re: [PATCH [uploaded to Rietveld]] Automatic lyric extenders, Trevor Daniels, 2016/12/27
- Re: [PATCH [uploaded to Rietveld]] Automatic lyric extenders, Alexander Kobel, 2016/12/27
- Re: [PATCH [uploaded to Rietveld]] Automatic lyric extenders, James, 2016/12/28
- Re: [PATCH [uploaded to Rietveld]] Automatic lyric extenders, Knut Petersen, 2016/12/25
Re: [PATCH] Automatic lyric extenders, Werner LEMBERG, 2016/12/22
Re: [PATCH] Automatic lyric extenders, Trevor Daniels, 2016/12/22