[Top][All Lists]

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

bug#23794: Emacs 25.0.94: Patch to make sort-lines respect visible lines

From: Eli Zaretskii
Subject: bug#23794: Emacs 25.0.94: Patch to make sort-lines respect visible lines (fairly urgent)
Date: Sat, 18 Jun 2016 21:19:36 +0300

> Date: Sat, 18 Jun 2016 20:49:12 +0300
> From: Eli Zaretskii <address@hidden>
> Cc: address@hidden
> > For clarity, the original behavior of sort-lines is what the patch 
> > restores. The backward-incompatibility to which
> > you refer is then just an implementation error that occurred when switching 
> > over to the overlay implementation
> > of outlines as there was never any documentation that I can see that 
> > suggested any behavior change. There
> > certainly could be better documentation as to whether a 'line' refers to a 
> > visible line, an invisible line or both
> > but many functions do not delineate this. A major reason for making lines 
> > invisible is so that they are not
> > treated as regular lines when functions are applied to buffer text. Thus, 
> > sort-lines should by default operate on
> > visible lines. It could be extended or another function could be written to 
> > operate on invisible lines as well, e.g.
> > sort-invisible-lines and an alias could be made to sort-lines to be called 
> > sort-visible-lines. All of this in the
> > future. The only thing I am suggesting for right now is to restore the 
> > original behavior. Note that if all lines are
> > visible, the patch codes works as well. The issue is that when lines are 
> > invisible the current code in Emacs
> > does not work in a very useful way.
> I think I already responded to this argument in my previous message.

In case it wasn't clear, I would welcome a change that is specific to
outline modes, whereby sorting would produce the same effect as it did
in Emacs 21 in these modes.  But even in outline modes, I think there
should be a way of getting the old behavior back.

reply via email to

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