[PUSHED][3-5] fdo#42865, SVN 1.7, etc.

Petr Mladek pmladek at suse.cz
Wed May 23 06:44:22 PDT 2012


Michael Stahl píše v St 23. 05. 2012 v 15:08 +0200:
> On 23/05/12 14:56, Petr Mladek wrote:
> 
> > I have only removed the trailing "\" from the last line of PATCH_FILES.
> > It worked because the next line was empty but I preferred to do not
> > concatenate it ;-)
> 
> actually i have over time come to prefer to add a trailing \ to the last
> line in such cases, and an empty line afterwards, because then appending
> a new line does not require changing the previous line to add a trailing
> \ which looks like spurious noise in diffs.

Makes sense. I am not sure that I like it. IMHO, it is less intuitive.
Removing the empty line might cause surprises. Well, it is mater of
taste. I do not have any strong opinion. I will leave it next time :-)


Best Regards,
Petr



More information about the LibreOffice mailing list