`pretty-print' by default quotes its output, so that `eval' can yield the
correct value. This is wrong in macro-expansions though, since the quote
wouldn't be there in the actual expanded code, and is also inconsistent with
other REPLs (e.g Guile).

Use a quote-depth of 1, which doesn't cause the extra quote to be printed for
macro expansions (`read' yields the correct value).

* elisp/geiser-repl.el(geiser-repl--sentinel): Use `substitute-command-keys'.

- fix typo that renamed the regex from "module" to "define-module"
- revert previous change that removed support for single-identifier module
  names
- as suggested, remove looking-at micro-optimisation, since it make code harder
  to read and doesn't seem to bring any relevant benefits.

Problem: when running geiser-compile-current-buffer (C-c C-k) on a file
using R7RS's define-library form, geiser-chicken--get-module starts an
infinite recursion. This happens because the original regular expression
contains two groups, but the code only tries to fetch the library name
from the first one with "(match-string-no-properties 1)".

Solution: we use the same strategy as in geiser-guile.el, namely we split the
regular expression into two separate ones, and check them one by one.

Jonas Bernoulli authored Oct 02, 2020 and jao committed Dec 02, 2020

Also add a FIXME comment about how this macro isn't actually needed.

Jonas Bernoulli authored Oct 02, 2020 and jao committed Dec 02, 2020

The whole first sentence should fit on the first line.  If that makes
the line a bit long then that is unfortunate but better than wrapping
it onto a new line.  When wrapping onto a new line anyway then the
second line should never be intended.  When it can be avoided, then
long first lines should be made shorter.

Jonas Bernoulli authored Oct 02, 2020 and jao committed Dec 02, 2020

This makes it possible to re-evaluate the containing
buffers without user customizations being clobbered.

Retrieve the scheme binary and arglist before switching to the repl
buffer, so that we respect any local settings from the source buffer.

Fixes an issue where chicken 5 users were taken to the chicken 4 external docs for the given query. With this update, users will be taken to chicken 5 or 4 docs based on the result of (geiser-chicken--version).

When evaluating expressions in a Scheme buffer, display the error
of an evaluation in the minibuffer regardless of whether
`geiser-debug-show-debug-p` or `geiser-debug-jump-to-debug-p` are
set or not.