[slime-devel] Daily ChangeLog diff

Index: slime/ChangeLog diff -u slime/ChangeLog:1.1260 slime/ChangeLog:1.1263 --- slime/ChangeLog:1.1260 Sat Dec 22 08:24:49 2007 +++ slime/ChangeLog Wed Jan 2 11:02:47 2008 @@ -1,3 +1,29 @@ +2008-01-02 Luís Oliveira <loliveira@common-lisp.net> + + Use sane default values for slime-repl-set-package. + + Previously, when typing `,!p' at the REPL, the current package + would have been inserted as a default (although the whole intent + was to /change/ the current package in the first place), now + nothing is inserted anymore. + + * slime.el (slime-pretty-current-package): rename it to + slime-pretty-find-buffer-package and make it use + slime-find-buffer-package instead of slime-current-package. + (slime-repl-set-package, slime-set-package): use new function. + +2008-01-02 Tobias C. Rittweiler <tcr@freebits.de> + + * slime.el (slime-print-apropos): Simplified: Don't insert action + properties anymore for the symbol; they were ignored anyway, + because `apropos-follow' (bound to RET in the resulting + *SLIME Apropos* buffer) looks for buttons only. + +2008-01-02 Tobias C. Rittweiler <tcr@freebits.de> + + * slime.el (slime-apropos): Update docstring: Apropos doesn't + match on regular expressions anymore since 2007-11-24. + 2007-12-22 Douglas Crosher <dcrosher@common-lisp.net> * swank-scl.lisp (set-stream-timeout, make-socket-io-stream): update

mbaringer@common-lisp.net (Marco Baringer) writes:
+2008-01-02 Luís Oliveira <loliveira@common-lisp.net> + + Use sane default values for slime-repl-set-package. + + Previously, when typing `,!p' at the REPL, the current package + would have been inserted as a default (although the whole intent + was to /change/ the current package in the first place), now + nothing is inserted anymore.
Hmm, I don't recall writing this explanation which is a good one except the last bit is not true. The patch reverts to the original behaviour of calling slime-find-buffer-package. I always use this command through the REPL buffer; indeed nothing is inserted in that case but that's not what happens if this command is invoked in another buffer. That might be a good thing for slime-repl-set-package since the repl's package doesn't necessarily match the buffer's. For slime-set-package, I don't know; I don't actually understand what it's used for.
+ * slime.el (slime-pretty-current-package): rename it to + slime-pretty-find-buffer-package and make it use + slime-find-buffer-package instead of slime-current-package. + (slime-repl-set-package, slime-set-package): use new function.
-- Luís Oliveira http://student.dei.uc.pt/~lmoliv/

Luis Oliveira <luismbo@gmail.com> writes:
+ [...] when typing `,!p' at the REPL [...]
Ugh, I missed this bit. Please ignore (most of) my previous message. -- Luís Oliveira http://student.dei.uc.pt/~lmoliv/

Luis Oliveira <luismbo@gmail.com> writes:
mbaringer@common-lisp.net (Marco Baringer) writes:
+2008-01-02 Luís Oliveira <loliveira@common-lisp.net> + + Use sane default values for slime-repl-set-package. + + Previously, when typing `,!p' at the REPL, the current package + would have been inserted as a default (although the whole intent + was to /change/ the current package in the first place), now + nothing is inserted anymore.
Hmm, I don't recall writing this explanation [...]
I added that to describe the purpose of the change. -T.
participants (3)
-
Luis Oliveira
-
mbaringer@common-lisp.net
-
Tobias C. Rittweiler