bug#23842: 24.4; Runaway background process

[Ken Brown <kbrown@cornell.edu>]

On 6/27/2016 11:49 AM, Eli Zaretskii wrote:
>> From: Ken Brown <kbrown@cornell.edu>
>> Date: Mon, 27 Jun 2016 09:24:06 -0400
>> Cc: 23842@debbugs.gnu.org
>>
>>      ;; Show possible completions in a temporary buffer.
>>      (with-output-to-temp-buffer "*Completions*"
>>        (display-completion-list allcomp))
>>      ;; Wait for a key press. Then delete *Completion*  window
>>      (momentary-string-display "" (point))
>>      (delete-window (get-buffer-window (get-buffer "*Completions*")))))
>>
>> It's hard to see how this could possibly work.  As soon as you click in
>> the *Completion* window, the window is deleted.
>
> It could perhaps work if we didn't check the window for being a live
> one.
>
> Anyway, the code is definitely broken and should be fixed.

Yes.  And there are even problems with it unrelated to the check for the 
window being live.  For example, suppose you want to switch to the 
*Completions* window to select the desired completion without using the 
mouse.  As soon as you type C-x (meant to be followed by o), the 
*Completions* window is gone.

Ken