[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH] target-mips: Remove semicolon from macro defini
From: |
Peter Maydell |
Subject: |
Re: [Qemu-devel] [PATCH] target-mips: Remove semicolon from macro definition |
Date: |
Sat, 22 Dec 2012 13:23:20 +0000 |
On 22 December 2012 12:38, Stefan Weil <address@hidden> wrote:
> Macro RESTORE_FLUSH_MODE is similar to RESTORE_ROUNDING_MODE
> but included a semicolon.
>
> The code which uses that macro also includes a semicolon,
> so the result was an empty statement.
>
> Remove the superfluous semicolon from the macro definition.
>
> Signed-off-by: Stefan Weil <address@hidden>
FWIW I think it would be better to just turn these two macros into
static inline functions which took a CPUMIPSState *env argument.
Macros which don't take arguments but silently use variables from
their caller are a bit ugly, and in this case there's no need to do it
that way.
-- PMM