qemu-devel
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [PULL 09/35] readline: Extract readline_add_completion_of() from mon


From: Philippe Mathieu-Daudé
Subject: Re: [PULL 09/35] readline: Extract readline_add_completion_of() from monitor
Date: Wed, 8 Feb 2023 12:06:09 +0100
User-agent: Mozilla/5.0 (Macintosh; Intel Mac OS X 10.15; rv:102.0) Gecko/20100101 Thunderbird/102.7.2

On 8/2/23 11:53, Markus Armbruster wrote:
Philippe Mathieu-Daudé <philmd@linaro.org> writes:

On 3/2/23 09:45, Markus Armbruster wrote:
monitor/misc.h has static add_completion_option().  It's useful
elsewhere in the monitor.  Since it's not monitor-specific, move it to
util/readline.c renamed to readline_add_completion_of(), and put it to
use.

52f50b1e9f8fd410d4293a211d549ec61b902728 is the first bad commit

This commit broke monitor autocomplete feature.

Before:

(qemu) <tab>
?                                announce_self
balloon                          block_job_cancel
block_job_complete               block_job_pause
block_job_resume                 block_job_set_speed
[...]

After:

(qemu) <tab>
[no autocomplete]

I'll look into it.  Thanks!

This fixed it:
20230207045241.8843-1-dongli.zhang@oracle.com/">https://lore.kernel.org/qemu-devel/20230207045241.8843-1-dongli.zhang@oracle.com/




reply via email to

[Prev in Thread] Current Thread [Next in Thread]