Re: [PATCH] build: Added -UNDEBUG to tools/Android.mk

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



> -----Original Message-----
> From: intel-gfx-bounces@xxxxxxxxxxxxxxxxxxxxx [mailto:intel-gfx-
> bounces@xxxxxxxxxxxxxxxxxxxxx] On Behalf Of devon.davies@xxxxxxxxx
> Sent: Wednesday, January 08, 2014 4:07 PM
> To: intel-gfx@xxxxxxxxxxxxxxxxxxxxx
> Cc: Davies, Devon
> Subject:  [PATCH] build: Added -UNDEBUG to tools/Android.mk
> 
> From: Devon Davies <devon.davies@xxxxxxxxx>
> 
> NDEBUG is defined by default in Android and causes assert(x) to be defined as
> void(x). This change ensures that asserts work as expected in tools.
> 
> This flag is already inside tests/Android.mk

I discussed and agreed to Devon´s solution with him before, so:
Reviewed-by: Oscar Mateo <oscar.mateo@xxxxxxxxx>

> Signed-off-by: Devon Davies <devon.davies@xxxxxxxxx>
> ---
>  tools/Android.mk |    2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/tools/Android.mk b/tools/Android.mk index 4a19b6c..5bda489
> 100644
> --- a/tools/Android.mk
> +++ b/tools/Android.mk
> @@ -40,7 +40,7 @@ define add_tool
> 
>      LOCAL_CFLAGS += -DHAVE_TERMIOS_H
>      LOCAL_CFLAGS += -DHAVE_STRUCT_SYSINFO_TOTALRAM
> -    LOCAL_CFLAGS += -DANDROID
> +    LOCAL_CFLAGS += -DANDROID -UNDEBUG
>      LOCAL_CFLAGS += -std=c99
>      # FIXME: drop once Bionic correctly annotates "noreturn" on pthread_exit
>      LOCAL_CFLAGS += -Wno-error=return-type
> --
> 1.7.9.5
> 
> ---------------------------------------------------------------------
> Intel-gfx mailing list
> Intel-gfx@xxxxxxxxxxxxxxxxxxxxx
> http://lists.freedesktop.org/mailman/listinfo/intel-gfx
_______________________________________________
Intel-gfx mailing list
Intel-gfx@xxxxxxxxxxxxxxxxxxxxx
http://lists.freedesktop.org/mailman/listinfo/intel-gfx





[Index of Archives]     [Linux USB Devel]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]
  Powered by Linux