[Openais] [PATCH] LINT: tweek the lint make rules.

Steven Dake sdake at redhat.com
Mon Nov 10 21:57:50 PST 2008


Good for merge.

Great work Angus!
-steve

On Mon, 2008-11-10 at 09:20 +1300, angus salkeld wrote:
> Makefile.inc: add -unrecog (don't complain about things like
>               strdup().
> tools/Makefile: don't use *.c as it complains about duplicate main()
>                 functions
> 
> Signed-off-by: Angus Salkeld <angus.salkeld at alliedtelesis.co.nz>
> ---
>  Makefile.inc   |    2 +-
>  tools/Makefile |    6 +++++-
>  2 files changed, 6 insertions(+), 2 deletions(-)
> 
> diff --git a/Makefile.inc b/Makefile.inc
> index 15f5ee5..3aeee4c 100644
> --- a/Makefile.inc
> +++ b/Makefile.inc
> @@ -46,7 +46,7 @@ endif
>  CFLAGS =
>  LDFLAGS =
>  DYFLAGS =
> -LINT_FLAGS = -weak +posixlib +ignoresigns -fcnuse -badflag -D__gnuc_va_list=va_list -D__attribute\(x\)= 
> +LINT_FLAGS = -weak -unrecog +posixlib +ignoresigns -fcnuse -badflag -D__gnuc_va_list=va_list -D__attribute\(x\)= 
>  
>  override CFLAGS += -DLCRSODIR='"$(LCRSODIR)"'
>  
> diff --git a/tools/Makefile b/tools/Makefile
> index 82888f3..029c8e0 100644
> --- a/tools/Makefile
> +++ b/tools/Makefile
> @@ -67,7 +67,11 @@ clean:
>  	rm -f *.o $(BINARIES)
>  
>  lint:
> -	-splint $(LINT_FLAGS) $(CFLAGS) *.c
> +	-splint $(LINT_FLAGS) $(CFLAGS) corosync-objctl.c
> +	-splint $(LINT_FLAGS) $(CFLAGS) corosync-cfgtool.c
> +	-splint $(LINT_FLAGS) $(CFLAGS) corosync-keygen.c
> +	-splint $(LINT_FLAGS) $(CFLAGS) corosync-fplay.c
> +	-splint $(LINT_FLAGS) $(CFLAGS) corosync-pload.c
>  
>  %.o: %.c
>  	$(CC) $(CFLAGS) $(CPPFLAGS) $(EXTRA_CFLAGS) -c -o $@ $<



More information about the Openais mailing list