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

Lubos Lunak l.lunak at suse.cz
Thu Mar 21 10:04:43 PDT 2013


On Thursday 21 of March 2013, 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 think there is still so much to do before arguing about spaces or other 
similar matter of taste details that have very little practical consequences.

-- 
 Lubos Lunak
 l.lunak at suse.cz


More information about the LibreOffice mailing list