* sly-stickers-tests.el (stickers-record-stuff): Call
`sly-stickers-fetch-all-and-forget'.

* contrib/sly-stickers.el (sly-stickers-mode-map): Change
`sly-sticker-dwim' binding yet again to C-c C-s C-s to free up C-c
C-s prefix for other sticker-related operations. Bind C-c C-s C-r
to unimplemented `sly-stickers-fetch-and-replay'.
(sly-stickers-fetch-all-and-forget): Renamed from
`sly-stickers-fetch'.
(sly-stickers-fetch-and-replay): New function.

* contrib/slynk-stickers.lisp (call-with-sticker-recording): New helper function.
(record): Use CALL-WITH-STICKER-RECORDING
(check-and-forget): Renamed from CHECK-STICKERS.

* sly-stickers.el (sly-stickers--decrease-prio)
(sly-stickers--increase-prio): Don't recurse.

* sly-stickers.el (sly-stickers-max-nested-stickers): New
user-visible customization variable.
(sly-stickers-color-face-attribute): New user-visible variable.
(sly-stickers--guess-face-color): New helper.
(sly-stickers--set-face): Use new helpers and variables.

* sly-stickers.el (sly-stickers-after-buffer-compilation): Use
`cl-loop' not `loop'

* sly-retro.el (sly-retro-slynk-to-swank): `sexp' may be a dotted
list, so don't use `mapcar'

In REVERT-GLOBAL-IO-REDIRECTION and related function, must be sure to
not signal errors. In this case SLYNK:*LOG-OUTPUT* must be used
instead of *STANDARD-OUT*, since the standard streams whose
redirection are about to be reverted might be in an unconsistent state
after, for instance, restarting an image.

* slynk-mrepl.lisp (open-dedicated-output-stream): Improve doc slightlyt.
(revert-global-io-redirection): Log to SLYNK:*LOG-OUTPUT*.

Some multithread implementations like Allegro don't see special
variables like *EMACS-CONNECTION* in new spawned threads.

If *USE-DEDICATED-OUTPUT-STREAM* is nil, this results in a grey stream
whose function fails when called from these threads. Otherwise the
dedicated output socket works OK.

This is **not** a concurrency issue, since that is handled by
SEND-TO-EMACS fine.

Test case for Allegro CL, which should work either in the main or
secondary REPLs.

   (setq slynk-mrepl:*use-dedicated-output-stream* nil)
   (mp:process-run-function "yo" (lambda ()
      (loop repeat 5 do (princ "Hello, world") (terpri) (sleep 1))))

* contrib/slynk-mrepl.lisp (make-mrepl-output-stream): Use
SLYNK:MAKE-THREAD-BINDINGS-AWARE-LAMBDA.
(:slynk-mrepl): Export *USE-DEDICATED-OUTPUT-STREAM*,
*DEDICATED-OUTPUT-STREAM-PORT*, and
*DEDICATED-OUTPUT-STREAM-BUFFERING*

* slynk/slynk.lisp (make-thread-bindings-aware-lambda): New function.
(:slynk-api): Export MAKE-THREAD-BINDINGS-AWARE-LAMBDA.

* slynk-mrepl.lisp (open-dedicated-output-stream): On
implementations not supporting :LINE as a value of
*USE-DEDICATED-OUTPUT-STREAM*, use an additional grey stream.

(sly-dispatch-event): Pass :RAISE to `sly--open-inspector'.

* sly-stickers.el (sly-stickers-fetch): Always provide some
feedback when fetching stickers.

This is a tentative change for various reasons:

- In slynk/slynk-gray.lisp a similar technique to be use here (git
  8a91fe75 Nikodemus Siivola 2006-04-12 08:43:55 +0000 32) but was
  removed in 2008. Bringing it back seems harmless enough, but only in
  limited tests. Also STREAM-WRITE-STRING does not implement flushing
  on newline.

- Some Lisps completely ignore the BUFFERING arg to ACCEPT-CONNECTION.

- All this is trying to avoid a dedicated thread for flushing all the
  output streams, which, being a dirty hack, might in the end be a
  the most battle-tested alternative.

Also the default value for SWANK-MREPL:*USE-DEDICATED-OUTPUT-STREAM*
should probably be moved to a function that detects if connection is
between the same host or not. This is actually a separate problem,
but detected it while editing the manual.

* contrib/slynk-mrepl.lisp (*dedicated-output-stream-buffering*): Default
to :line, not t. SBCL and CMUCL seem to abide. CCL and ALLEGRO
ignore it.

* doc/sly.texi (REPL output control): New section.

* slynk/slynk-gray.lisp (stream-write-char): For the STREAM-WRITE-CHAR
interface, FINISH-OUTPUT when a newline is detected.

* doc/sly.texi: Remove sub menus that make it very hard to edit and
restructure the manual.

* slynk/slynk-loader.lisp (src-files): Guard against case where SRC-DIR does not 
have a pathname directory.

* slynk-mrepl.lisp (sync-package-and-default-directory)
(guess-and-set-package): Slave "slyfuns" must return an elisp-
readable value.

* CONTRIBUTING.md: Update example and fix a typo.

* contrib/sly-mrepl.el (sly-mrepl--result-counter): Remove variable.
(sly-mrepl-mode): Don't set `sly-mrepl--result-counter'
(:write-values): Rename arg to "results".
(:clear-repl-history): Don't set `sly-mrepl--result-counter'.
(sly-mrepl--eval-for-repl): Rewrite using `cl-destructuring-bind'.
(sly-mrepl--make-result-button): Only two arguments.
(sly-mrepl--insert-returned-values): Don't use `sly-mrepl--result-counter'.
(sly-mrepl-shortcut-alist): Add "clear repl" as a REPL shortcut.
* test/sly-stickers-tests.el (stickers-in-a-file): New test.
(stickers-record-stuff): Fix FIXME.

* contrib/slynk-mrepl.lisp (make-results): New helper.
(mrepl-eval): Use MAKE-RESULTS.
(inspect-entry): Don't use SLYNK::TO-LINE here.
(copy-to-repl): Use MAKE-RESULTS.

explicitly to appease sbcl.

* sly.el (sly-region-for-defun-at-point): Accept POS as default point.

* contrib/sly-stickers.el
(sly-stickers-compile-region-aware-of-stickers-1): New helper
function. Accept `callback'. Accept `sync', `flash` and `fallback`
as boolean kwargs.
(sly-stickers-compile-region-aware-of-stickers): Make use of
`sly-stickers-compile-region-aware-of-stickers-1'
(sly-stickers-after-buffer-compilation): Implemented, makes use of
`sly-stickers-compile-region-aware-of-stickers-1'.

* contrib/slynk-stickers.lisp (defpackage :slynk-stickers): Export slyfuns
explicitly to appease sbcl.
(compile-for-stickers): Add docstring. Only return PROBE if it was
a successful compilation. Return nil if no ORIGINAL-STRING was
given.

* test/sly-stickers-tests.el (stickers-in-a-file): New test.
(stickers-record-stuff): Fix FIXME.

* sly.el (sly-menu): Reorganize. Tracing options moved to
`sly-trace-dialog' contrib.

* contrib/sly-trace-dialog.el (sly-trace-dialog-untrace-all): New function.
(sly-trace-dialog--shortcut-menu): Add new "Untrace all" option.

* contrib/slynk-trace-dialog.lisp (dialog-untrace-all): Return number of
specs untraced.

* sly.el (sly-messages): Require new lib.
(sly-message, sly--last-message, sly-temp-message, sly-warning)
(sly-error, sly-display-oneliner, sly-oneliner)
(sly-completing-read-function, sly-completing-read)
(sly-flash-inhibit, sly-flash-region): Move to new
lib/sly-messages.el lib

* lib/sly-buttons.el (sly-messages): require new lib.

* lib/sly-messages.el: New file.

* sly-buttons.el (sly-button--searchable-buttons-at): Ensure the
non-overlay button is actually not an overlay.

Include the docstring in the inspector for SBCL

Update refcard docs and cross-reference sections

Update the keymaps in the documentation and cross-reference sections
of the refcard to match the current default maps in the code.

An idea by henkvr from https://github.com/slime/slime/issues/193.

* slynk/backend/slynk-sbcl.lisp (emacs-inspect): Include the
  documentation string of the function in the inspector for the SBCL
  backend.

* sly.el (sly-macroexpansion-minor-mode-map): Use defvar.
(sly-show-notes): Use `sly-button-flash'.

* contrib/sly-mrepl.el (sly-mrepl-mode-map): Define more interesting
bindings for M-p and M-n, also C-M-p, C-M-n.
(sly-mrepl--make-result-button): Assign `sly-button-search-id' to
new button.
(sly-mrepl-previous-input-or-button)
(sly-mrepl-next-input-or-button): New functions.

* contrib/sly-stickers.el (sly-stickers--echo-sticker): Use new `sly-button-flash'
(sly-stickers--flash-sticker): Removed.
(sly-stickers-fetch): Use new `sly-button-flash'

* lib/sly-buttons.el (sly-button-at): Use `button-type-subtype-p'
(sly-button): Set a basic `sly-button-search-id' here.
(sly-button-flash): New function.
(sly-button-echo-part): Flash buttons by default.
(sly-button--overlays-starting-at): Renamed to
`sly-button--searchable-buttons-starting-at'
(sly-button--searchable-buttons-at): Consider text buttons.
(sly-button-search): Use `sly-button--searchable-buttons-at'
(sly-button-search): Error instead of message.

* test/sly-mrepl-tests.el (sly-mrepl-tests--assert-prompt): More
interesting search for prompt.
(sly-mrepl-tests--with-basic-repl-setup): New macro.
(repl-values-and-button-navigation): New test.

* sly-stickers.el (sly-stickers-after-buffer-compilation): Check for `stickers'

* sly.el (sly-macroexpansion-minor-mode-map): Use defvar.

* contrib/sly-mrepl.el (sly-mrepl-shortcut-key): Move down to "comma shortcut" section.
(sly-mrepl-reset-shortcut-key): Check for `sly-mrepl-shortcut-key' with boundp.

Also make the macroexpansion buffer read-only.

* sly.el (remap): Remove obsolete toplevel remappings block.
(sly-macroexpansion-minor-mode-map): Setup remaps here.

* sly-mrepl.el (sly-mrepl-shortcut-key): Add a `:set'
specification, otherwise the variable's useless.
(sly-mrepl-reset-shortcut-key): New function.
(sly-mrepl--shortcut-location-p): Only attempt to search for "`"
if the shortcut key is ","

* contrib/sly-mrepl.el (sly-mrepl--eval-for-repl): Use `after-prompt'.
(sly-mrepl--eval-for-repl): Accept `before-prompt' and
`after-prompt' callbacks.
(sly-mrepl--copy-objects-to-repl): Pass `callback' as an
`after-prompt' callback.
(sly-mrepl--insert-call): Fix off-by-one.

* contrib/sly-mrepl.el (sly-mrepl--copy-to-repl-after): Removed
hackish variable.
(sly-mrepl-mode): Ditto.
(:copy-to-repl): Delete channel method.
(sly-mrepl--insert-prompt): `condition' arg is optional.
(sly-mrepl--eval-for-repl): New function (little relation to the
old function with the same name.)
(sly-mrepl--copy-objects-to-repl): Use `sly-mrepl--eval-for-mrepl'
(sly-mrepl--save-and-copy-for-repl): Renamed from
`sly-mrepl--eval-for-mrepl'.
(sly-inspector-copy-part-to-repl, sly-db-copy-part-to-repl)
(sly-apropos-copy-symbol-to-repl)
(sly-trace-dialog-copy-part-to-repl, sly-db-copy-call-to-repl)
(sly-trace-dialog-copy-call-to-repl, sly-mrepl-set-directory): Use
`sly-mrepl--save-and-copy-for-repl'.
(sly-mrepl-set-package): Use `sly-mrepl--eval-for-repl'.

* contrib/slynk-mrepl.lisp (copy-values-to-repl): Deleted.
(prompt-arguments): New helper.
(send-prompt): Use PROMPT-ARGUMENTS.
(:copy-to-repl): Removed channel method.
(:sync-package-and-default-directory): Removed channel method.
(guess-and-set-package): Moved slyfun down.
(eval-for-mrepl): Add docstring and clarify behaviour.
(copy-to-repl, sync-package-and-default-directory): New "slave" slyfuns.

* contrib/sly-mrepl.el (sly-mrepl-shortcut-key): New variable.
(sly-mrepl-mode-map): New entry for `sly-mrepl-shortcut-key'
(sly-mrepl--shortcut-location-p): New helper.
(sly-mrepl-shortcut-alist): New variable.
(sly-mrepl-set-package, sly-mrepl-set-directory): New functions.
(sly-mrepl-shortcut): New interactive command.

* contrib/slynk-mrepl.lisp (:sync-package-and-default-directory):
Use new GUESS-AND-SET-PACKAGE.
(guess-and-set-package): New slyfun.
(with-eval-for-repl): Accept an extra MREPL-SYM arg
(eval-for-mrepl): Send prompt after evaluating slave slyfun.

If point is at note beginning and last command is not a navigation
command, begin by displaying the note at point. Otherwise go on to
next note as usual. Affects `sly-button-forward' and
`sly-button-backward'.

* sly-buttons.el (sly-button--overlays-starting-at): New helper.
(sly-button--last-search-command): New variable.
(sly-button--search-1): New helper.
(sly-button-search): Use `sly-button--search-1' and
`sly-button--overlays-starting-at'