Subject: bug#73330: closed (Re: bug#73330: 31.0.50; Incorrect completions for 'cond' clauses)

From: [email-masked] (GNU bug Tracking System)
Date: Thu, 19 Sep 2024 16:39:03 +0000

Your bug report

#73330: 31.0.50; Incorrect completions for 'cond' clauses

which was filed against the emacs package, has been closed.

The explanation is attached below, along with your original report.
If you require more details, please reply to [email-masked].

-- 
73330: https://debbugs.gnu.org/cgi/bugreport.cgi?bug=73330
GNU Bug Tracking System
Contact [email-masked] with problems

Subject: Re: bug#73330: 31.0.50; Incorrect completions for 'cond' clauses
From: Eli Zaretskii <[email-masked]>
To: Eshel Yaron <[email-masked]>
Cc: [email-masked]
Date: Thu, 19 Sep 2024 19:38:12 +0300
List-Id: <debbugs-submit.debbugs.gnu.org>

> From: Eshel Yaron <[email-masked]>
> Cc: [email-masked]
> Date: Thu, 19 Sep 2024 16:33:11 +0200
> 
> Eli Zaretskii <[email-masked]> writes:
> 
> > Does "helps" mean the original scenario is fixed? or are there any
> > leftovers in that scenario?
> 
> Yes, it fixed the original scenario.
> 
> >> but also seems to cause a regression.  With this change I get:
> >> 
> >> 1. emacs -Q
> >> 2. Insert "(cond ((current-p"
> >>                   ^note the additional parenthesis
> >> 3. Hit M-TAB to complete
> >> 
> >> This shows an opposite bug: Emacs should complete function names (and
> >> without the change, it does), but with the change we now get variable
> >> name completion.
> >
> > How about the patch below?
> 
> That works well in both the original scenario and the modified one.

Thanks, installed on master, and closing the bug.

Subject: 31.0.50; Incorrect completions for 'cond' clauses
From: Eshel Yaron <[email-masked]>
To: [email-masked]
Date: Wed, 18 Sep 2024 14:36:18 +0200
List-Id: <debbugs-submit.debbugs.gnu.org>

Hi,

1. emacs -Q
2. Insert "(cond (current-p
3. Hit M-TAB to complete

We're completing a symbol in the condition of a cond clause, so it is
expected to be completed as a variable, to current-prefix-argument.
However, Emacs mistakenly thinks that we're completing a function name,
and completes to current-bidi-paragraph-direction.

Thanks,

Eshel