[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH] ui/gtk: Allow user to select monitor number to display qemu
From: |
Markus Armbruster |
Subject: |
Re: [PATCH] ui/gtk: Allow user to select monitor number to display qemu in full screen through new gtk display option |
Date: |
Mon, 14 Jun 2021 17:00:56 +0200 |
User-agent: |
Gnus/5.13 (Gnus v5.13) Emacs/27.2 (gnu/linux) |
Eric Blake <eblake@redhat.com> writes:
> On Mon, Jun 14, 2021 at 04:05:42PM +0800, sweeaun wrote:
>> -display gtk,monitor-num=<value>
>>
>> Signed-off-by: sweeaun <swee.aun.khor@intel.com>
>> ---
>> qapi/ui.json | 3 ++-
>> qemu-options.hx | 2 +-
>> ui/gtk.c | 15 +++++++++++++++
>> 3 files changed, 18 insertions(+), 2 deletions(-)
>>
>> diff --git a/qapi/ui.json b/qapi/ui.json
>> index 1052ca9c38..c3363c7ad5 100644
>> --- a/qapi/ui.json
>> +++ b/qapi/ui.json
>> @@ -1041,7 +1041,8 @@
>> ##
>> { 'struct' : 'DisplayGTK',
>> 'data' : { '*grab-on-hover' : 'bool',
>> - '*zoom-to-fit' : 'bool' } }
>> + '*zoom-to-fit' : 'bool',
>> + '*monitor-num' : 'int' } }
>
> Missing documentation of the new member. Also, get rid of the tab damage.
Suggest to name it either @monitor-number or @monitor (because what else
than the number could 'int' be?)