[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:26:54 +0100 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:17.0) Gecko/20130110 Thunderbird/17.0.2 |
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)?
Paolo
- [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, 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",
Paolo Bonzini <=
- Re: [Qemu-devel] [PATCH] gtk: Replace bogus term "VGA" with "display", Paolo Bonzini, 2013/02/22