Re: [PATCH] NULL: remove other definition of NULL

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

 



On Wed, 13 Apr 2016 20:11:33 +0800
Du Huanpeng <d-hp@xxxxxxx> wrote:

> From: Du Huanpeng <u74147@xxxxxxxxx>
> 
> remove other definition of NULL, use
>   #include <linux/stddef.h>
> instead.
> I use this command to search NULL definition,
>   grep -R "define\s*\<NULL\>"
> hope there are no more definition of NULL.
> 
> Signed-off-by: Du Huanpeng <u74147@xxxxxxxxx>
> ---
>  include/command.h   |  5 +----
>  lib/bzlib_private.h | 11 +----------
>  2 files changed, 2 insertions(+), 14 deletions(-)
> 
> diff --git a/include/command.h b/include/command.h
> index e41068a..b938431 100644
> --- a/include/command.h
> +++ b/include/command.h
> @@ -25,10 +25,7 @@
>  
>  #include <linux/list.h>
>  #include <linux/stringify.h>
> -
> -#ifndef NULL
> -#define NULL	((void *)0)
> -#endif
> +#include <linux/stddef.h>

It looks like this chunk removes lines introduced in your previous 'NULL: define as a null pointer'
patch.

Can we join your patches?

>  #ifndef	__ASSEMBLY__
>  
> diff --git a/lib/bzlib_private.h b/lib/bzlib_private.h
> index 87d8f94..de10dd4 100644
> --- a/lib/bzlib_private.h
> +++ b/lib/bzlib_private.h
> @@ -67,6 +67,7 @@
>  #define _BZLIB_PRIVATE_H
>  
>  #include <malloc.h>
> +#include <linux/stddef.h>
>  
>  #include "bzlib.h"
>  
> @@ -515,16 +516,6 @@ BZ2_hbCreateDecodeTables ( Int32*, Int32*, Int32*, UChar*,
>  
>  #endif
>  
> -
> -/*-- BZ_NO_STDIO seems to make NULL disappear on some platforms. --*/
> -
> -#ifdef BZ_NO_STDIO
> -#ifndef NULL
> -#define NULL 0
> -#endif
> -#endif
> -
> -
>  /*-------------------------------------------------------------*/
>  /*--- end                                   bzlib_private.h ---*/
>  /*-------------------------------------------------------------*/
> -- 
> 1.9.1
> 
> 
> 
> _______________________________________________
> barebox mailing list
> barebox@xxxxxxxxxxxxxxxxxxx
> http://lists.infradead.org/mailman/listinfo/barebox


-- 
-- 
Best regards,
  Antony Pavlov

_______________________________________________
barebox mailing list
barebox@xxxxxxxxxxxxxxxxxxx
http://lists.infradead.org/mailman/listinfo/barebox




[Index of Archives]     [Linux Embedded]     [Linux USB Devel]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]     [XFree86]

  Powered by Linux