[slime-devel] Daily ChangeLog diff
Index: slime/ChangeLog diff -u slime/ChangeLog:1.1890 slime/ChangeLog:1.1896 --- slime/ChangeLog:1.1890 Tue Oct 27 08:46:32 2009 +++ slime/ChangeLog Wed Oct 28 16:52:20 2009 @@ -1,3 +1,41 @@ +2009-10-28 Helmut Eller <heller@common-lisp.net> + + Simpler modeline code. + + * slime.el (slime-modeline-string): Renamed from + slime-compute-modeline-string. + (slime-modeline-state-string): Renamed from + slime-compute-connection-state + (slime-modeline-package, slime-modeline-connection-name) + (slime-modeline-connection-state) + (slime-extended-modeline,slime-compute-modeline-package) + (slime-update-modeline-string, slime-shall-we-update-modeline-p) + (slime-update-all-modelines, slime-modeline-update-timer) + (slime-restart-or-init-modeline-update-timer) + (slime-connection-state-as-string): Deleted + (slime-stale-connection-p): Deleted. + +2009-10-28 Helmut Eller <heller@common-lisp.net> + + * slime.el (slime-disconnect): Don't reference connection. Left + over from last change. + ([test] arglist): Update expected results for slightly changed + printer settings. + ([test] indentation): Install common-lisp-indent-function. + ([undefun] display-warning): Fix it. + + * test.sh: Don't copy contribs. Slime should work without them. + * swank-loader.lisp (setup): Compile contribs only if directory + exists. + +2009-10-28 Helmut Eller <heller@common-lisp.net> + + * slime.el (slime-format-display-message, slime-display-message) + (slime-create-message-window): Deleted. The trick with the + pre-command-hook doesn't work in XEmacs 21.5. So use the standard + message function. One day XEmacs will learn how to resize the + minibuffer, but until then we have to live with one-line messages. + 2009-10-27 Helmut Eller <heller@common-lisp.net> * slime.el: Fix some docstrings and comments.
Helmut Eller <heller@common-lisp.net> writes:
+2009-10-28 Helmut Eller <heller@common-lisp.net> + + * slime.el (slime-disconnect): Don't reference connection. Left + over from last change. + ([test] arglist): Update expected results for slightly changed + printer settings. + ([test] indentation): Install common-lisp-indent-function.
Regarding indentation, I long wanted to make slime-setup set the lisp-*-loop-indentation variables to saner default values. The default values that Emacs ships with are made for inserting a newline after the LOOP symbol, i.e. (loop for x from ... for y from ...) Reading code, most people (including me) are more fond of beginning the first loop clause following the LOOP symbol, i.e. (loop for x from ... for y from ...) To get that indentation, one have to use (setq lisp-simple-loop-indentation 2 lisp-loop-keyword-indentation 6 lisp-loop-forms-indentation 6) How controversial is it to make that the default values for slime-mode? -T.
* Tobias C. Rittweiler [2009-10-29 10:17+0100] writes:
Regarding indentation, I long wanted to make slime-setup set the lisp-*-loop-indentation variables to saner default values.
The default values that Emacs ships with are made for inserting a newline after the LOOP symbol, i.e.
(loop for x from ... for y from ...)
Reading code, most people (including me) are more fond of beginning the first loop clause following the LOOP symbol, i.e.
(loop for x from ... for y from ...)
To get that indentation, one have to use
(setq lisp-simple-loop-indentation 2 lisp-loop-keyword-indentation 6 lisp-loop-forms-indentation 6)
How controversial is it to make that the default values for slime-mode?
It would be better to discuss this with the maintainers of lisp-mode. Slime isn't the place to fix lisp-mode. Helmut
* Tobias C. Rittweiler [2009-10-29 10:27+0100] writes:
Helmut Eller <heller@common-lisp.net> writes:
It would be better to discuss this with the maintainers of lisp-mode. Slime isn't the place to fix lisp-mode.
It's supposed to be "superior". I don't want to fix lisp-mode, but have good default values for slime-mode.
Well, superior than ilisp or inf-lisp or eli. lisp-mode is in a different realm. Helmut
On 29 Oct 2009, at 09:17, Tobias C. Rittweiler wrote:
Regarding indentation, I long wanted to make slime-setup set the lisp-*-loop-indentation variables to saner default values.
The default values that Emacs ships with are made for inserting a newline after the LOOP symbol, i.e.
(loop for x from ... for y from ...)
Reading code, most people (including me) are more fond of beginning the first loop clause following the LOOP symbol, i.e.
(loop for x from ... for y from ...)
On a related note, is it possible to improve the indentation of grouped loop clauses? For example: (loop for cited-title in (gather-referrenced-titles entries) for target-entries = (gethash cited-title title/entry-hash) unless target-entries collect cited-title into orphans else when (> (length target-entries) 1) collect cited-title into ambiguities finally (return (values orphans ambiguities)))) OTOH perhaps I overuse LOOP ;-) Further examples (including nested clauses, although I rarely/never use those): <http://www.lispworks.com/documentation/HyperSpec/Body/ 06_aha.htm> -- Duncan
* Duncan Harvey [2009-10-31 17:32+0100] writes:
On a related note, is it possible to improve the indentation of grouped loop clauses? For example:
(loop for cited-title in (gather-referrenced-titles entries) for target-entries = (gethash cited-title title/entry-hash) unless target-entries collect cited-title into orphans else when (> (length target-entries) 1) collect cited-title into ambiguities finally (return (values orphans ambiguities))))
contrib/slime-indentation.el supposedly does that. Helmut
participants (3)
-
Duncan Harvey -
Helmut Eller -
Tobias C. Rittweiler