Age | Commit message (Collapse) | Author |
|
- Avoid defining autoload definitions in a central location.
Instead add autoload cookies to the forms/definitions that should
be autoloaded, in the locations where the actual definitions are
located.
- Do this for `geiser-mode', `turn-on-geiser-mode',
`geiser-mode--maybe-activate' (including adding that to
`scheme-mode-hook'), `geiser', `geiser-connect',
`geiser-connect-local' and `geiser-repl-switch'.
- Also do this for `run-geiser', even though it is only an obsolete
function alias for `geiser', which might make it desirable to drop
the autoload altogether.
Some unusual autoload definitions remain in "geiser.el", see below.
- One issue with defining autoloads in a central location is that it
is easy to forget to remove such autoloads when the real definition
is removed.
No longer autoload `geiser-version' because since [1: 847d2ad]
there no longer exists a proper definition of that function.
- No longer autoload `geiser-unload', `geiser-reload' and
`turn-off-geiser-mode', because they are only useful if Geiser has
already been loaded, at which point any autoloaded definitions are
no longer relevant.
However,
- Keep autoloading `geiser-activate-implementation' and
`geiser-implementation-extension', even though I doubt that this
is actually useful.
- Keep using `custom-add-load' to specify dependencies of Custom
groups and keep autoloading that. I don't know if this is actually
necessary, and while it seems really weird, it might served a legit
purpose, that I am not aware of.
1: 2020-07-19 847d2ad4c6da462c26c50af1ef7d9cd697f3a5d2
scheme and autotools removals
|
|
In Emacs 30:
lib/geiser/elisp/geiser-compile.el:19:11: Warning:
‘make-network-process´ called without required keyword argument :service
This seems to be a false-positive, but I am not sure.
|
|
|
|
The bytecode compiler complains because delete-backward-char is an interactive
function.
|
|
|
|
Disabled by default. Adds new custom variables
"geiser-repl-superparen-character" and "geiser-repl-superparen-mode-p".
|
|
|
|
"geiser-repl-autoeval-delay" should be "geiser-repl-autoeval-mode-delay".
|
|
Disabled by default. Adds new custom variables
"geiser-repl-autoeval-mode-delay" and "geiser-repl-autoeval-mode-p".
|
|
Should provide a better fix for extended issue #58.
|
|
This should take care of the problem reported in issue #58.
|
|
* geiser-compile-file
* geiser-compile-current-buffer
* geiser-load-current-buffer
* geiser-add-to-load-path
|
|
Fixes #57.
|
|
|
|
|
|
|
|
Should address issue #55 for all implementations.
|
|
|
|
|
|
|
|
|
|
|
|
|
|
Some authors of downstream packages are reluctant to do stuff with
upstream packages' private functions.
(for example, https://github.com/abo-abo/swiper/pull/2235#issuecomment-537399457)
|
|
Should address, sort of, issue #46
|
|
Thanks to Brian Leung
|
|
|
|
See discussion in issue #48.
|
|
Should fix issue #48
|
|
|
|
NOTE: The patch is largely untested.
Modifications:
- Update readme.org
- Remove geiser-company
- Move Company extensions to geiser-completion
Omissions:
- geiser-company--inhibit-autodoc has been removed. Eldoc handling
should be implemented in the frontend, not in the backend.
See for example:
https://github.com/minad/corfu/blob/04fbfce3d7e9c125a7fd22a34455a508247a522b/corfu.el#L1212
- The quickhelp-string action and geiser-company--docstring have been
removed. company-quickhelp can use `:company-doc-buffer` instead with
minimal overhead.
See:
https://github.com/company-mode/company-quickhelp/blob/3ca2708b4e5190205aca01d65fe1b391963a53f9/company-quickhelp.el#L138
- The automatic Company setup has been removed. Personally I am not a
fan of such auto configuration. It is better if completion is
configured consistently in the user configuration. You may want to
restore the auto configuration for backward compatibility. It depends
on your backward compatibility story. I am fine with rare breaking
changes from time to time.
- There is a cyclic dependency between geiser-edit/geiser-doc and
geiser-completion, which should be untangled.
|
|
So that now all start with *Geiser (and a space for hidden ones) and
use consistent capitalization (see issue #38).
|
|
Fixes issue #37.
|
|
|
|
|
|
|
|
|
|
Thanks to Denis Golovachev for pointing this out!
|
|
Like quitting a REPL right after opening it, or calling evaluation
functions in the wrong, non-scheme buffer.
|
|
|
|
|
|
|
|
And we still need some fixes here when that's active, specifically for
the case when you travel to a file outside the current project: it
becomes repeless, which is a sad state.
|
|
|
|
|
|
Together with project-current as a default for the project root
function when available, now that Emacs 28 is on the horizon.
|
|
|
|
|
|
|
|
FSVO better, but at least the code is cleaner and prompts regexps more
accurate.
|