|
From: | Avi Kivity |
Subject: | Re: [Qemu-devel] [PATCH] vga: squash build error in vga_update_memory_access() |
Date: | Wed, 24 Aug 2011 15:58:37 +0300 |
User-agent: | Mozilla/5.0 (X11; Linux x86_64; rv:6.0) Gecko/20110816 Thunderbird/6.0 |
On 08/24/2011 03:46 PM, Jan Kiszka wrote:
> +++ b/hw/vga.c > @@ -179,6 +179,8 @@ static void vga_update_memory_access(VGACommonState *s) > base = 0xb8000; > size = 0x8000; > break; > + default: > + abort(); > } > region = g_malloc(sizeof(*region)); > memory_region_init_alias(region, "vga.chain4",&s->vram, offset, size); ...or just make the last case default?
No reason to make the code unobvious in this path, IMO. Eventually gcc will be able to drop the 4/5 bytes this patch adds to the object code.
-- I have a truly marvellous patch that fixes the bug which this signature is too narrow to contain.
[Prev in Thread] | Current Thread | [Next in Thread] |