Skip site navigation (1)Skip section navigation (2)
Date:      Sat, 13 Nov 2010 10:15:00 -0500
From:      George Neville-Neil <gnn@FreeBSD.org>
To:        Mikolaj Golub <to.my.trociny@gmail.com>
Cc:        svn-src-head@freebsd.org, svn-src-all@freebsd.org, src-committers@freebsd.org
Subject:   Re: svn commit: r215207 - in head: sys/net sys/netinet tools/regression/netinet/arphold
Message-ID:  <C902813A-6C82-4395-BFEC-FDD1617B29D0@FreeBSD.org>
In-Reply-To: <86fwv57fph.fsf@kopusha.home.net>
References:  <201011122203.oACM32cN024939@svn.freebsd.org> <86fwv57fph.fsf@kopusha.home.net>

next in thread | previous in thread | raw e-mail | index | archive | help


On Nov 13, 2010, at 04:09 , Mikolaj Golub wrote:

> 
> On Fri, 12 Nov 2010 22:03:02 +0000 (UTC) George V. Neville-Neil wrote:
> 
> GVN> Author: gnn
> GVN> Date: Fri Nov 12 22:03:02 2010
> GVN> New Revision: 215207
> GVN> URL: http://svn.freebsd.org/changeset/base/215207
> 
> 
> 
> cc -c -O -pipe  -std=c99 -g -Wall -Wredundant-decls -Wnested-externs -Wstrict-prototypes  -Wmissing-prototypes -Wpointer-arith -Winline -Wcast-qual  -Wundef -Wno-pointer-sign -fformat-extensions -nostdinc  -I. -I/usr/src/sys -I/usr/src/sys/contrib/altq -D_KERNEL -DHAVE_KERNEL_OPTION_HEADERS -include opt_global.h -fno-common -finline-limit=8000 --param inline-unit-growth=100 --param large-function-growth=1000  -mno-align-long-strings -mpreferred-stack-boundary=2 -mno-mmx -mno-3dnow -mno-sse -mno-sse2 -mno-sse3 -ffreestanding -fstack-protector -Werror  /usr/src/sys/net/if_llatbl.c
> cc1: warnings being treated as errors
> /usr/src/sys/net/if_llatbl.c: In function 'llentry_free':
> /usr/src/sys/net/if_llatbl.c:124: warning: format '%ld' expects type 'long int', but argument 4 has type 'size_t'
> *** Error code 1
> 
> I think this should be:
> 
> Index: sys/net/if_llatbl.c
> ===================================================================
> --- sys/net/if_llatbl.c	(revision 215233)
> +++ sys/net/if_llatbl.c	(working copy)
> @@ -122,7 +122,7 @@ llentry_free(struct llentry *lle)
> 	}
> 
> 	KASSERT(lle->la_numheld == 0, 
> -		("%s: la_numheld %d > 0, pkts_droped %ld", __func__, 
> +		("%s: la_numheld %d > 0, pkts_droped %zu", __func__, 
> 		 lle->la_numheld, pkts_dropped));
> 
> 	LLE_FREE_LOCKED(lle);

Looks like Konstatin already fixed this.

Best,
George




Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?C902813A-6C82-4395-BFEC-FDD1617B29D0>