Re: [elpa] 01/01: * company-capf.el (company--capf-data): Don't get confused by lambda exps.

Re: [elpa] 01/01: * company-capf.el (company--capf-data): Don't get confused by lambda exps.

[Dmitry Gutov]

Stefan Monnier <monnier-CRDzTM1onBSWkKpYnGOUKg@public.gmane.org> writes:

>            (data (run-hook-wrapped 'completion-at-point-functions
>                                    ;; Ignore misbehaving functions.
>                                    #'completion--capf-wrapper 'optimist)))
> -    (when (consp data) data)))
> +    (when (and (consp data) (numberp (car data))) data)))

This will always fail because completion--capf-wrapper returns the
completion function itself in the car. START and END are at positions 1
and 2. I'll push a fixed version shortly.

Re: [elpa] 01/01: * company-capf.el (company--capf-data): Don't get confused by lambda exps.

[Stefan Monnier]

>> (data (run-hook-wrapped 'completion-at-point-functions
>> ;; Ignore misbehaving functions.
>> #'completion--capf-wrapper 'optimist)))
>> -    (when (consp data) data)))
>> +    (when (and (consp data) (numberp (car data))) data)))
> This will always fail because completion--capf-wrapper returns the
> completion function itself in the car.  START and END are at positions 1
> and 2. I'll push a fixed version shortly.

Duh!  Thanks for catching this.  I see now that I "tested" it without
recompiling, which is why it seemed to work as well as before.


        Stefan