Skip site navigation (1)Skip section navigation (2)
Date:      Sun, 21 Jun 2009 08:49:06 +0000 (UTC)
From:      Roman Divacky <rdivacky@FreeBSD.org>
To:        src-committers@freebsd.org, svn-src-all@freebsd.org, svn-src-head@freebsd.org
Subject:   svn commit: r194577 - head/sys/net
Message-ID:  <200906210849.n5L8n6Ki000741@svn.freebsd.org>

next in thread | raw e-mail | index | archive | help
Author: rdivacky
Date: Sun Jun 21 08:49:06 2009
New Revision: 194577
URL: http://svn.freebsd.org/changeset/base/194577

Log:
  In non-debugging mode make this define (void)0 instead of nothing. This
  helps to catch bugs like the below with clang.
  
  	if (cond);		<--- note the trailing ;
  	   something();
  
  Approved by:	ed (mentor)
  Discussed on:	current@

Modified:
  head/sys/net/bridgestp.c

Modified: head/sys/net/bridgestp.c
==============================================================================
--- head/sys/net/bridgestp.c	Sun Jun 21 08:36:30 2009	(r194576)
+++ head/sys/net/bridgestp.c	Sun Jun 21 08:49:06 2009	(r194577)
@@ -68,7 +68,7 @@ __FBSDID("$FreeBSD$");
 #ifdef	BRIDGESTP_DEBUG
 #define	DPRINTF(fmt, arg...)	printf("bstp: " fmt, ##arg)
 #else
-#define	DPRINTF(fmt, arg...)
+#define	DPRINTF(fmt, arg...)	(void)0
 #endif
 
 #define	PV2ADDR(pv, eaddr)	do {		\



Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?200906210849.n5L8n6Ki000741>