Re: [Qemu-devel] [PATCH 06/24] Remove local ram_size that hides global one

[Alexander Graf]

On 13.11.2012, at 13:11, Christian Borntraeger wrote:

> From: Heinz Graalfs <address@hidden>
> 
> The global variable 'ram_size' is hidden by the local variable
> declaration in s390_init()

That's the point of Eduardo's patch, no? Or do we need access to the global to 
change its value afterwards? If so, please write a reasonable patch description 
that actually explains the problem.


Alex

> 
> Signed-off-by: Heinz Graalfs <address@hidden>
> Signed-off-by: Christian Borntraeger <address@hidden>
> ---
> hw/s390-virtio.c |    1 -
> 1 file changed, 1 deletion(-)
> 
> diff --git a/hw/s390-virtio.c b/hw/s390-virtio.c
> index ebec844..78477af 100644
> --- a/hw/s390-virtio.c
> +++ b/hw/s390-virtio.c
> @@ -155,7 +155,6 @@ unsigned s390_del_running_cpu(CPUS390XState *env)
> static void s390_init(QEMUMachineInitArgs *args)
> {
>     ram_addr_t my_ram_size = args->ram_size;
> -    ram_addr_t ram_size = args->ram_size;
>     const char *cpu_model = args->cpu_model;
>     const char *kernel_filename = args->kernel_filename;
>     const char *kernel_cmdline = args->kernel_cmdline;
> -- 
> 1.7.10.1
>