[PATCH] remove/add blank lines in sw/source/core

Kohei Yoshida kohei.yoshida at gmail.com
Thu Mar 21 09:46:32 PDT 2013


On 03/21/2013 09:31 AM, Eike Rathke wrote:
> Hi Philipp,
>
> On Wednesday, 2013-03-20 17:31:03 +0000, Philipp Riemer (via Code Review) wrote:
>
>> remove/add blank lines in sw/source/core
>
> I'd prefer if such mere cosmetical changes would not be done; while
> inserting a blank line mostly is ok for readability, removing a blank
> line mostly for the same reason decreases readability. After all it
> boils down to personal preferences. i.e. personally I usually use two
> blank lines between method implementations, just because browsing the
> source on a quick fly in the editor is easier for the eye.
>
> How do others see this?

I personally prefer a single blank line.  Having two blank lines doesn't 
really improve readability for me, actually.

Kohei

-- 
Kohei Yoshida, LibreOffice hacker, Calc


More information about the LibreOffice mailing list