all messages for Emacs-related lists mirrored at yhetil.org
 help / color / mirror / code / Atom feed
* lisp-completion question
@ 2013-06-26 11:39 Thierry Volpiatto
  2013-07-04  0:24 ` Stefan Monnier
  0 siblings, 1 reply; 7+ messages in thread
From: Thierry Volpiatto @ 2013-06-26 11:39 UTC (permalink / raw)
  To: emacs-devel

Hi,

It seem `lisp-completion-at-point' is no more returning :predicate,
(accessible in emacs-24.3 with (plist-get plist :predicate))
is it wanted?
Note: `lisp-complete-symbol' is still using this with no effect of course. 

In `lisp--local-variables' e.g &optional is returned, is it wanted?
otherwise, the patch below fix it:

diff --git a/lisp/emacs-lisp/lisp.el b/lisp/emacs-lisp/lisp.el
index b37a811..af8433a 100644
--- a/lisp/emacs-lisp/lisp.el
+++ b/lisp/emacs-lisp/lisp.el
@@ -725,7 +725,7 @@ considered."
         (delq nil
               (mapcar (lambda (var)
                         (and (symbolp var)
-                             (not (string-match (symbol-name var) "\\`[&_]"))
+                             (not (string-match "\\`[&_]" (symbol-name var)))
                              ;; Eliminate uninterned vars.
                              (intern-soft var)
                              var))


Thanks.

-- 
Thierry
Get my Gnupg key:
gpg --keyserver pgp.mit.edu --recv-keys 59F29997 




^ permalink raw reply related	[flat|nested] 7+ messages in thread

* Re: lisp-completion question
  2013-06-26 11:39 lisp-completion question Thierry Volpiatto
@ 2013-07-04  0:24 ` Stefan Monnier
  2013-07-04  6:04   ` Thierry Volpiatto
  0 siblings, 1 reply; 7+ messages in thread
From: Stefan Monnier @ 2013-07-04  0:24 UTC (permalink / raw)
  To: Thierry Volpiatto; +Cc: emacs-devel

> It seem `lisp-completion-at-point' is no more returning :predicate,
> (accessible in emacs-24.3 with (plist-get plist :predicate))
> is it wanted?

It sometimes does, sometimes doesn't.  Do you see a difference in the
resulting behavior?

> In `lisp--local-variables' e.g &optional is returned, is it wanted?

No.

> otherwise, the patch below fix it:
> diff --git a/lisp/emacs-lisp/lisp.el b/lisp/emacs-lisp/lisp.el
> index b37a811..af8433a 100644
> --- a/lisp/emacs-lisp/lisp.el
> +++ b/lisp/emacs-lisp/lisp.el
> @@ -725,7 +725,7 @@ considered."
>          (delq nil
>                (mapcar (lambda (var)
>                          (and (symbolp var)
> -                             (not (string-match (symbol-name var) "\\`[&_]"))
> +                             (not (string-match "\\`[&_]" (symbol-name var)))
>                               ;; Eliminate uninterned vars.
>                               (intern-soft var)
>                               var))

Very much so, indeed, thanks for catching this typo.  Feel free to
install that patch,


        Stefan



^ permalink raw reply	[flat|nested] 7+ messages in thread

* Re: lisp-completion question
  2013-07-04  0:24 ` Stefan Monnier
@ 2013-07-04  6:04   ` Thierry Volpiatto
  2013-07-04  9:58     ` Stefan Monnier
  0 siblings, 1 reply; 7+ messages in thread
From: Thierry Volpiatto @ 2013-07-04  6:04 UTC (permalink / raw)
  To: Stefan Monnier; +Cc: emacs-devel

Hello Stefan,

Stefan Monnier <monnier@IRO.UMontreal.CA> writes:

>> It seem `lisp-completion-at-point' is no more returning :predicate,
>> (accessible in emacs-24.3 with (plist-get plist :predicate))
>> is it wanted?
>
> It sometimes does, sometimes doesn't.  Do you see a difference in the
> resulting behavior?

Yes, for me here it is never returned and as result, (predicate is nil)
I get all the unbounded symbol in the resulting lists, so I dont use
anymore this function, I must use my own completing function with a
predicate determined by a function based on emacs-24.3 code.

> Very much so, indeed, thanks for catching this typo.  Feel free to
> install that patch,

Please, do it, I don't use anymore bzr.

Thanks.

-- 
Thierry
Get my Gnupg key:
gpg --keyserver pgp.mit.edu --recv-keys 59F29997 



^ permalink raw reply	[flat|nested] 7+ messages in thread

* Re: lisp-completion question
  2013-07-04  6:04   ` Thierry Volpiatto
@ 2013-07-04  9:58     ` Stefan Monnier
  2013-07-04 11:11       ` Thierry Volpiatto
  0 siblings, 1 reply; 7+ messages in thread
From: Stefan Monnier @ 2013-07-04  9:58 UTC (permalink / raw)
  To: Thierry Volpiatto; +Cc: emacs-devel

> I get all the unbounded symbol in the resulting lists,

Sounds like a bug, indeed.  Can you give me a concrete example?

>> Very much so, indeed, thanks for catching this typo.  Feel free to
>> install that patch,
> Please, do it, I don't use anymore bzr.

Presumably you can now use Git to access the Bzr repository.


        Stefan



^ permalink raw reply	[flat|nested] 7+ messages in thread

* Re: lisp-completion question
  2013-07-04  9:58     ` Stefan Monnier
@ 2013-07-04 11:11       ` Thierry Volpiatto
  2013-07-04 11:18         ` Thierry Volpiatto
  2013-07-04 18:14         ` Stefan Monnier
  0 siblings, 2 replies; 7+ messages in thread
From: Thierry Volpiatto @ 2013-07-04 11:11 UTC (permalink / raw)
  To: Stefan Monnier; +Cc: emacs-devel

Stefan Monnier <monnier@IRO.UMontreal.CA> writes:

>> I get all the unbounded symbol in the resulting lists,
>
> Sounds like a bug, indeed.  Can you give me a concrete example?

;; emacs-24.3

helm!
M-: (lisp-completion-at-point)
=>[...]
    :predicate
    #[(sym)
      "\301\b!\206

^ permalink raw reply	[flat|nested] 7+ messages in thread

* Re: lisp-completion question
  2013-07-04 11:11       ` Thierry Volpiatto
@ 2013-07-04 11:18         ` Thierry Volpiatto
  2013-07-04 18:14         ` Stefan Monnier
  1 sibling, 0 replies; 7+ messages in thread
From: Thierry Volpiatto @ 2013-07-04 11:18 UTC (permalink / raw)
  To: emacs-devel

Sorry, the previous mail was truncated.

;; emacs-24.3

helm!
M-: (lisp-completion-at-point)
=>[...]
    :predicate
    #[(sym)
      "\301\b!\206.




^ permalink raw reply	[flat|nested] 7+ messages in thread

* Re: lisp-completion question
  2013-07-04 11:11       ` Thierry Volpiatto
  2013-07-04 11:18         ` Thierry Volpiatto
@ 2013-07-04 18:14         ` Stefan Monnier
  1 sibling, 0 replies; 7+ messages in thread
From: Stefan Monnier @ 2013-07-04 18:14 UTC (permalink / raw)
  To: Thierry Volpiatto; +Cc: emacs-devel

> helm!
> M-: (lisp-completion-at-point)
> =>[...]
>     :predicate
>     #[(sym)
>       "\301\b!\206\302\b!\206\303\b!\207"
>       [sym boundp fboundp symbol-plist]
>       2]

No, I really mean a concrete example of difference in the resulting set
of candidates.  Whether a predicate is used or not is an internal detail.


        Stefsan



^ permalink raw reply	[flat|nested] 7+ messages in thread

end of thread, other threads:[~2013-07-04 18:14 UTC | newest]

Thread overview: 7+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2013-06-26 11:39 lisp-completion question Thierry Volpiatto
2013-07-04  0:24 ` Stefan Monnier
2013-07-04  6:04   ` Thierry Volpiatto
2013-07-04  9:58     ` Stefan Monnier
2013-07-04 11:11       ` Thierry Volpiatto
2013-07-04 11:18         ` Thierry Volpiatto
2013-07-04 18:14         ` Stefan Monnier

Code repositories for project(s) associated with this external index

	https://git.savannah.gnu.org/cgit/emacs.git
	https://git.savannah.gnu.org/cgit/emacs/org-mode.git

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.