simple-completions now completes package names as well

[lispegistus]

A small improvement to simple completion to include package names as well. For example the completion for: alex<tab> will be alexandria: and alexandria-2: instead of getting a "No completions found" message.

[joaotavora]

Lispegistus @.***> writes:

A small improvement to simple completion to include package names as well. For example the completion for: alex will be alexandria: and alexandria-2: instead of getting a "No completions found" message.

If you can confirm this is how it works in SLIME, I don't see much of a issue in adopting something like this

But there's something I'm not crazy about, which is that you don't consider the current value of PACKAGE in your patch. It is available as the PACKAGE argument to the function you changed.

In my opinion, it doesn't make much sense to offer completions for package prefixes that are already accessible in the current context, either because they are PACKAGE or PACKAGE :USEs them.

You also don't consider package nicknames, right? How does that work? What about package-local nicknames? :-)

João

[lispegistus]

Thank you for the feedback. Indeed, package nicknames is something I need to think about as well as the state of the current package. I'll look into how slime does things in more detail and rework this when I have a bit more time.