[Fwd: Re: [slime-devel] Re: slime.el `slime-expand-abbreviations-and-complete']

[enometh at meer.net]

I know I said I didn't mind if this change was revered, but in the
interest of correctness, I think the code Edi checked in again is
plain wrong. Details below.


---------------------------- Original Message ----------------------------
Subject: Re: [slime-devel]  Re: slime.el
`slime-expand-abbreviations-and-complete' From:    "Edi Weitz"
<edi at agharta.de>
Date:    Sat, January 20, 2007 2:43
To:      enometh at meer.net
--------------------------------------------------------------------------

On 20 Jan 2007 10:34:55 -0000, Madhu <enometh at meer.net> wrote:

>   |However, I regularly use slime-complete-symbol* as my completion
|function (but with other pop-up settings) and my text never got
|garbled.  And I liked the behaviour where point was directly
>   |after the shortest unambiguous prefix.
>
> With slime-complete-symbol* and a vanilla checkout, text gets
> garbled again on emacs21. Perhaps the property setting shennanigans dont
work on this emacs?
>
> I think you are have not considered all use cases and are also
> ignoring the fact that if you are using emacs' `choose-completion'
facility, you are not allowed to shift the point to an earlier
> position during the completion operation.
>
> If you do wish to move the point, you should perhaps automatically
invoke the completion command programatically again. I am not sure how
this can be done.
>
> Even if it WFM, and provides a shiny feature, it should be clear that
the existing code is wrong?

I'd rather have this discussion on slime-devel if it's really
necessary.