slime.el cleanup and a minor improvement
João Távora
joaotavora at gmail.com
Sat Mar 8 14:01:49 UTC 2014
Hi,
I'd like to push the three changes to SLIME's master branch. These
effectively mark the end of XEmacs compatibility and are mostly about
deleting code.
They live in my fork, git at github.com:capitaomorte/slime.git, in the the
`slime-el-cleanup' branch:
commit b9dfcba7d59c56e7b6b8d38413bd8f8801b13da3
Remove XEmacs portability layer and fix some FIXMEs
44 0 ChangeLog
108 632 slime.el
commit 92be40bcd47d640849f1df807d971cbe76b733c5
Redesign and simplify some window-managing code in slime.el
32 1 ChangeLog
80 212 slime.el
commit 8951befc40df6f693f0606789c616f312442992a
Improve window selection for popup buffers
Use Emacs 24.3's window-managing API to do less surprising choices
when popping up temporary SLIME buffers. This is innocuous on
previous Emacs versions.
12 0 ChangeLog
20 5 slime.el
The first is mostly a big cleanup of slime.el's XEmacs "portability
layer", fixes some FIXME's and uses "declare" specs for indentation. It
removes more than 500 lines in slime.el
The second simplifies the slime-popup-* logic, targets more FIXME's and
adapts some parts to more common idioms.
The third commit adds 15 lines that, in my opinion, greatly improve
window selection for popup buffers, but only on Emacs >= 24.3. They fix
a common debugger annoyance: When I have my frame divided into 3+
windows, selecting restarts that immediatly trigger another debugger
makes the SLDB buffer cycle between visible windows. This commit fixes
that and subsequent SLDB buffers go to the same window if possible.
The cleanup is relatively aggressive and, though neither I nor the
automated tests have noticed it, it's likely that I broke something that
I don't use regularly.
Please test and review my changes. Except for the third commit, no
functional changes should be observed. If you're acquainted with
slime.el, point to the offending code whenever possible. Otherwise
describe your use cases that were broken so I can fix it.
Thanks,
João
More information about the slime-devel
mailing list