Re: [Qemu-devel] [PATCH 1.1] tci: Fix wrong macro name for debug code

[Anthony Liguori]

On 05/03/2012 11:09 AM, Stefan Weil wrote:
> Code which is compiled with CONFIG_TCG_DEBUG (set by configure option
> --enable-debug-tcg) should not disable the assert macro by
> defining NDEBUG.
>
> With the wrong macro name CONFIG_TCG_DEBUG, all assertions in tci.c
> were completely useless because NDEBUG was always defined.
>
> Signed-off-by: Stefan Weil<address@hidden>

Applied.  Thanks.

Regards,

Anthony Liguori

> ---
>   tci.c |    2 +-
>   1 files changed, 1 insertions(+), 1 deletions(-)
>
> Please apply this fix before the release of QEMU 1.1.
>
> Thanks,
>
> Stefan Weil
>
> diff --git a/tci.c b/tci.c
> index 23a8368..e40c080 100644
> --- a/tci.c
> +++ b/tci.c
> @@ -20,7 +20,7 @@
>   #include "config.h"
>
>   /* Defining NDEBUG disables assertions (which makes the code faster). */
> -#if !defined(CONFIG_TCG_DEBUG)&&  !defined(NDEBUG)
> +#if !defined(CONFIG_DEBUG_TCG)&&  !defined(NDEBUG)
>   # define NDEBUG
>   #endif