[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH] gtk: Replace bogus term "VGA" with "display"
From: |
Jan Kiszka |
Subject: |
Re: [Qemu-devel] [PATCH] gtk: Replace bogus term "VGA" with "display" |
Date: |
Fri, 22 Feb 2013 15:29:24 +0100 |
User-agent: |
Mozilla/5.0 (X11; U; Linux i686 (x86_64); de; rv:1.8.1.12) Gecko/20080226 SUSE/2.0.0.12-1.1 Thunderbird/2.0.0.12 Mnenhy/0.7.5.666 |
On 2013-02-22 15:17, Anthony Liguori wrote:
> Jan Kiszka <address@hidden> writes:
>
>> On 2013-02-22 14:21, Anthony Liguori wrote:
>>> Hi Jan,
>>>
>>> Jan Kiszka <address@hidden> writes:
>>>
>>>> VGA is a misnomer, only some of our targets have it. Use the more
>>>> generic term "display" instead.
>>>>
>>>> Signed-off-by: Jan Kiszka <address@hidden>
>>>
>>> Sorry, I responded to your previous note in my head and forgot to also
>>> respond on the mailing list.
>>>
>>> We expose this "misnomer" a lot. For instance, qemu -vga cirrus.
>>
>> That's because qemu started for x86 only. No excuse to spread this
>> further in new code.
>>
>>>
>>> OTOH, the term "display" is used to indicate the backend via qemu
>>> -display sdl.
>>>
>>> So calling the VGA tab "Display" after selecting '-display gtk' seems a
>>> bit confusing to me.
>>>
>>> So I'm open to picking better naming but I think we should try to make
>>> it consistent throughout QEMU.
>>
>> We can also call it "screen" or "guest display" - just not VGA,
>> please.
>
> I don't care what we call it, I'm asking that if you want to change it
> in one place, change it every where else too.
For obvious reasons, -vga has to stay. Also, I can leave the code as is.
My primary point is the user-visible misnomer "VGA". That should be
fixed before rolling it out. I'll trim down my patch to this.
Jan
--
Siemens AG, Corporate Technology, CT RTC ITP SDP-DE
Corporate Competence Center Embedded Linux
- [Qemu-devel] [PATCH] gtk: Replace bogus term "VGA" with "display", Jan Kiszka, 2013/02/22
- Re: [Qemu-devel] [PATCH] gtk: Replace bogus term "VGA" with "display", Anthony Liguori, 2013/02/22
- Re: [Qemu-devel] [PATCH] gtk: Replace bogus term "VGA" with "display", Jan Kiszka, 2013/02/22
- Re: [Qemu-devel] [PATCH] gtk: Replace bogus term "VGA" with "display", Anthony Liguori, 2013/02/22
- Re: [Qemu-devel] [PATCH] gtk: Replace bogus term "VGA" with "display",
Jan Kiszka <=
- Re: [Qemu-devel] [PATCH] gtk: Replace bogus term "VGA" with "display", Anthony Liguori, 2013/02/22
- Re: [Qemu-devel] [PATCH] gtk: Replace bogus term "VGA" with "display", Jan Kiszka, 2013/02/22
- Re: [Qemu-devel] [PATCH] gtk: Replace bogus term "VGA" with "display", Paolo Bonzini, 2013/02/22
- Re: [Qemu-devel] [PATCH] gtk: Replace bogus term "VGA" with "display", Paolo Bonzini, 2013/02/22