Index: slime/ChangeLog diff -u slime/ChangeLog:1.1984 slime/ChangeLog:1.1987 --- slime/ChangeLog:1.1984 Tue Feb 16 06:28:18 2010 +++ slime/ChangeLog Wed Feb 17 12:04:59 2010 @@ -1,3 +1,58 @@ +2010-02-17 Helmut Eller heller@common-lisp.net + + Add a command to eval stuff in the inspector. + + * slime.el (slime-inspector-eval): New command. + (slime-inspector-mode-map): Bind it to 'e'. + * swank.lisp (inspector-eval): New function. + * swank-backend.lisp (eval-context): New function. + * swank-cmucl.lisp (eval-context): Implement it. + +2010-02-17 Helmut Eller heller@common-lisp.net + + Point-entered hooks in the xref buffer don't work so well. + Use forward/backward commands instead. + + * slime.el (slime-xref-next-line, slime-xref-prev-line): New + commands. + (slime-xref-show-location): New aux function. + (slime-xref-mode-map): Remap up/down to next/prev location + commands. Make SPC and RET do the same. + +2010-02-17 Helmut Eller heller@common-lisp.net + + Select the xref buffer. + I never could get used to the next/prev cycling. + + * slime.el (slime-with-xref-buffer): Select the buffer. + (slime-insert-xrefs): Add point-entered hook to automatically + display the current location. + (slime-xref-entered): New function. + (slime-show-buffer-position): Don't use reposition-window which + seems very slow and doesn't even do a particularly good job. The + new heuristic is much cruder but faster. + (slime-xref-mode-map): Remove apparently redundant bindings for + RET. + +2010-02-17 Helmut Eller heller@common-lisp.net + + Get rid or snapshots. + + * slime.el (slime-narrowing-configuration, slime-emacs-snapshot) + (slime-current-narrowing-configuration) + (slime-set-narrowing-configuration, slime-current-emacs-snapshot) + (slime-set-emacs-snapshot) + (slime-current-emacs-snapshot-fingerprint, slime-frame-windows): + Deleted. Update call-sites accordingly. + +2010-02-17 Helmut Eller heller@common-lisp.net + + Test suite hangs for CCL and CMUCL. + + * slime.el ([test] compile-defun): Remove those compile-time read + errors for which some implementations invoke the debugger. + (sexp-at-point.1): Remove failing tests + 2010-02-16 Tobias C. Rittweiler tcr@freebits.de
* swank-loader.lisp: Compile files on ECL, too. Index: slime/contrib/ChangeLog diff -u slime/contrib/ChangeLog:1.343 slime/contrib/ChangeLog:1.345 --- slime/contrib/ChangeLog:1.343 Mon Feb 15 16:42:37 2010 +++ slime/contrib/ChangeLog Wed Feb 17 12:04:59 2010 @@ -1,3 +1,10 @@ +2010-02-17 Helmut Eller heller@common-lisp.net + + Fix bugs when *inferior-buffers* doesn't exist. + + * inferior-slime.el (inferior-slime-start-transcript) + (inferior-slime-stop-transcript): Make sure the buffer exists. + 2010-02-15 Tobias C. Rittweiler tcr@freebits.de
* slime-asdf.el, slime-autodoc.el, slime-c-p-c.el, @@ -26,6 +33,13 @@ (slime-search-exports-in-defpackage): take #:symbol and :symbol into account too.
+2010-02-17 Helmut Eller heller@common-lisp.net + + Fix bugs when *inferior-buffers* doesn't exist. + + * inferior-slime.el (inferior-slime-start-transcript) + (inferior-slime-stop-transcript): Make sure the buffer exists. + 2010-01-14 Stas Boukarev stassats@gmail.com
* slime-repl.el: Revert the previous change because it
+2010-02-17 Helmut Eller heller@common-lisp.net + + Point-entered hooks in the xref buffer don't work so well. + Use forward/backward commands instead. + + * slime.el (slime-xref-next-line, slime-xref-prev-line): New + commands. + (slime-xref-show-location): New aux function. + (slime-xref-mode-map): Remap up/down to next/prev location + commands. Make SPC and RET do the same. +
1. 'n' and 'p' use the functions `next-line' and `previous-line', respectively. Are these used intentionally in slime code instead of (forward-line 1) and (forward-line -1)?
From the description of `next-line':
"If you are thinking of using this in a Lisp program, consider using `forward-line' instead. It is usually easier to use and more reliable (no dependence on goal column, etc.)."
From the description of `previous-line':
"If you are thinking of using this in a Lisp program, consider using `forward-line' with a negative argument instead. It is usually easier to use and more reliable (no dependence on goal column, etc.)."
2. If `previous-line' and `next-line' are not used intentionally, please find attached a patch that uses `forward-line'. It also:
- replaces the obsolete (alias) variable `last-command-char' with `last-command-event'
- deletes the call to the obsolete function `make-local-hook' in the function `slime-add-local-hook', and simplifies the code.
The patch is relative to slime.el, revision 1.1276
+2010-02-17 Helmut Eller heller@common-lisp.net + + Select the xref buffer. + I never could get used to the next/prev cycling. + + * slime.el (slime-with-xref-buffer): Select the buffer. + (slime-insert-xrefs): Add point-entered hook to automatically + display the current location. + (slime-xref-entered): New function.
3. There is no function named `slime-xref-entered' in slime.el, revision 1.1276. Was this unintentionally omitted? Typing C-c C-c in the ChangeLog yields the error message: Source location of tag `slime-xref-entered' not found in file `slime.el'. grep on *.el does not find this identifier, either.
Index: slime/contrib/ChangeLog diff -u slime/contrib/ChangeLog:1.343 slime/contrib/ChangeLog:1.345 --- slime/contrib/ChangeLog:1.343 Mon Feb 15 16:42:37 2010 +++ slime/contrib/ChangeLog Wed Feb 17 12:04:59 2010 @@ -1,3 +1,10 @@ +2010-02-17 Helmut Eller heller@common-lisp.net + + Fix bugs when *inferior-buffers* doesn't exist. + + * inferior-slime.el (inferior-slime-start-transcript) + (inferior-slime-stop-transcript): Make sure the buffer exists. +
4. This description was included in the contrib/Changelog twice. Was another description intended to be included instead?
---
* Mark Harig [2010-02-18 19:21+0100] writes:
- 'n' and 'p' use the functions `next-line' and `previous-line',
respectively. Are these used intentionally in slime code instead of (forward-line 1) and (forward-line -1)?
Yes that's intentional. The up/down cursor keys are usually bound to next-line resp. previous-line.
forward-line only searches the next newline, while next-line goes to the next line and additionally tries to preserve the current column among other things. forward-line is a primitive written in C and next-line runs a ton of Lisp code.
- If `previous-line' and `next-line' are not used intentionally,
please find attached a patch that uses `forward-line'. It also:
- replaces the obsolete (alias) variable `last-command-char' with
`last-command-event'
Does that work in XEmacs?
- deletes the call to the obsolete function `make-local-hook'
in the function `slime-add-local-hook', and simplifies the code.
The patch is relative to slime.el, revision 1.1276
+2010-02-17 Helmut Eller heller@common-lisp.net
- Select the xref buffer.
- I never could get used to the next/prev cycling.
- slime.el (slime-with-xref-buffer): Select the buffer.
- (slime-insert-xrefs): Add point-entered hook to automatically
- display the current location.
- (slime-xref-entered): New function.
- There is no function named `slime-xref-entered' in slime.el,
revision 1.1276. Was this unintentionally omitted? Typing C-c C-c in the ChangeLog yields the error message: Source location of tag `slime-xref-entered' not found in file `slime.el'. grep on *.el does not find this identifier, either.
It was there in 1.1274, but deleted in the next commit and I forgot to mention it. But you wouldn't find the deleted function anyway :-)
Index: slime/contrib/ChangeLog diff -u slime/contrib/ChangeLog:1.343 slime/contrib/ChangeLog:1.345 --- slime/contrib/ChangeLog:1.343 Mon Feb 15 16:42:37 2010 +++ slime/contrib/ChangeLog Wed Feb 17 12:04:59 2010 @@ -1,3 +1,10 @@ +2010-02-17 Helmut Eller heller@common-lisp.net
- Fix bugs when *inferior-buffers* doesn't exist.
- inferior-slime.el (inferior-slime-start-transcript)
- (inferior-slime-stop-transcript): Make sure the buffer exists.
- This description was included in the contrib/Changelog twice.
Was another description intended to be included instead?
No, I just botched the merge.
Thanks for the review :-) I committed some of your changes.
Helmut
- If `previous-line' and `next-line' are not used intentionally,
please find attached a patch that uses `forward-line'. It also:
- replaces the obsolete (alias) variable `last-command-char' with
`last-command-event'
Does that work in XEmacs?
I do not know whether `last-command-event' is included in XEmacs.
The documentation for `last-command-char' says that it has been obsolete since Emacs 19.34.
Perhaps someone who uses XEmacs could respond.
---
Index: slime/ChangeLog diff -u slime/ChangeLog:1.1984 slime/ChangeLog:1.1987 --- slime/ChangeLog:1.1984 Tue Feb 16 06:28:18 2010 +++ slime/ChangeLog Wed Feb 17 12:04:59 2010 @@ -1,3 +1,58 @@ +2010-02-17 Helmut Eller heller@common-lisp.net + + Add a command to eval stuff in the inspector. + + * slime.el (slime-inspector-eval): New command. + (slime-inspector-mode-map): Bind it to 'e'.
Please find attached a patch file for 'slime.texi' in the 'doc' directory that adds descriptive text to the 'Inspector' node for the new command and key that you have added.
Also, the introductory text in the Inspector node told the reader to see the generic function `inspect-for-emacs' in the file 'swank-backend.lisp'. There is no function with that name in that file. I changed the text to reference the generic function `emacs-inspect', instead.
I tested this change by generating new .info, .pdf, and HTML files from the new .texi file.
---
-
Helmut Eller -
Mark Harig