qemu-devel
[Top][All Lists]
Advanced

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

Re: [Qemu-devel] [PATCH] gtk: Replace bogus term "VGA" with "display"


From: Paolo Bonzini
Subject: Re: [Qemu-devel] [PATCH] gtk: Replace bogus term "VGA" with "display"
Date: Fri, 22 Feb 2013 16:29:04 +0100
User-agent: Mozilla/5.0 (X11; Linux x86_64; rv:17.0) Gecko/20130110 Thunderbird/17.0.2

Il 22/02/2013 16:26, Paolo Bonzini ha scritto:
> Il 22/02/2013 15:51, Anthony Liguori ha scritto:
>>>> For obvious reasons, -vga has to stay.
>> We can introduce a new option that aliases this the -vga option.
>>
>> [08:50 AM] address@hidden:~/git/qemu$ grep vga *.c | wc -l
>> 51
>>
>> This is not in the hw/ directory.  All of our internal infrastructure
>> calls it VGA.  It's odd to me to call it something different in the UI
>> without changing the internal bits too.
> 
> It's all in vl.c, and all about -vga.
> 
> $ grep -c vga vl.c
> 46
> 
> These are what are called vga_ by "mistake":
> 
> $ git grep -c vga origin/master -- ui/
> origin/master:ui/cocoa.m:2
> origin/master:ui/console.c:27
> origin/master:ui/curses.c:2
> origin/master:ui/gtk.c:18
> origin/master:ui/sdl.c:10
> origin/master:ui/spice-core.c:1
> origin/master:ui/spice-display.c:1
> origin/master:ui/vgafont.h:1
> origin/master:ui/vnc.c:5
> 
> What about:
> 
> - s/vga_hw_/guest_display_/ (e.g. guest_display_invalidate)
> 
> - "Guest" or "Display" or "Guest display" for the GTK UI (and similarly
> in the code, e.g. gd_on_guest_display)?

Saw Jan's v2, it looks like s/vga_hw_/screen_/ would also have no conflicts.

Paolo




reply via email to

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