emacs-devel
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [Emacs-diffs] emacs-24 r117265: Fix bug #17801 with extraneous newli


From: Eli Zaretskii
Subject: Re: [Emacs-diffs] emacs-24 r117265: Fix bug #17801 with extraneous newlines after inserting markup in Texinfo mode.
Date: Fri, 20 Jun 2014 22:24:14 +0300

> From: Stefan Monnier <address@hidden>
> Cc: address@hidden
> Date: Fri, 20 Jun 2014 11:50:40 -0400
> 
> >> > -  \n _ \n "@end " str \n)
> >> > +  \n _ \n "@end " str \n \n)
> >> I think these extra \n should not be added.
> > If you mean that "\n" should have been used instead, then I agree and
> > will change these.  But if you mean these skeletons should not insert
> > newlines unconditionally, then please explain, because I tested these
> > changes and found them to DTRT.
> 
> I think they should end with a single \n and nothing more, so they won't
> add a newline when there's one already.

I added another \n in 4 cases: texinfo-insert-block,
address@hidden, address@hidden, and
address@hidden  All of these should have an empty line
after them.  So I think I disagree with what you say, and these
changes are actually correct.

In any case, "at eol" and "there is already a newline" is not the
same.  skeleton.el tests the former, which doesn't mean there's a
newline.



reply via email to

[Prev in Thread] Current Thread [Next in Thread]