Skip site navigation (1)Skip section navigation (2)
Date:      Mon, 21 May 2012 02:40:03 GMT
From:      dfilter@FreeBSD.ORG (dfilter service)
To:        freebsd-bugs@FreeBSD.org
Subject:   Re: kern/168095: commit references a PR
Message-ID:  <201205210240.q4L2e34J049140@freefall.freebsd.org>

next in thread | raw e-mail | index | archive | help
The following reply was made to PR kern/168095; it has been noted by GNATS.

From: dfilter@FreeBSD.ORG (dfilter service)
To: bug-followup@FreeBSD.org
Cc:  
Subject: Re: kern/168095: commit references a PR
Date: Mon, 21 May 2012 02:30:35 +0000 (UTC)

 Author: kevlo
 Date: Mon May 21 02:30:22 2012
 New Revision: 235711
 URL: http://svn.freebsd.org/changeset/base/235711
 
 Log:
   Fix improper handling of variadic args with ICDEBUG
   
   PR:	kern/168095
   Submitted by:	gcooper
 
 Modified:
   head/sys/libkern/iconv.c
   head/sys/sys/iconv.h
 
 Modified: head/sys/libkern/iconv.c
 ==============================================================================
 --- head/sys/libkern/iconv.c	Mon May 21 02:23:20 2012	(r235710)
 +++ head/sys/libkern/iconv.c	Mon May 21 02:30:22 2012	(r235711)
 @@ -549,9 +549,7 @@ int
  iconv_lookupcp(char **cpp, const char *s)
  {
  	if (cpp == NULL) {
 -		ICDEBUG("warning a NULL list passed\n", ""); /* XXX ISO variadic								macros cannot
 -								leave out the
 -								variadic args */
 +		ICDEBUG("warning a NULL list passed\n", "");
  		return ENOENT;
  	}
  	for (; *cpp; cpp++)
 
 Modified: head/sys/sys/iconv.h
 ==============================================================================
 --- head/sys/sys/iconv.h	Mon May 21 02:23:20 2012	(r235710)
 +++ head/sys/sys/iconv.h	Mon May 21 02:30:22 2012	(r235711)
 @@ -240,7 +240,7 @@ int iconv_converter_tolowerstub(int c, v
  int iconv_converter_handler(module_t mod, int type, void *data);
  
  #ifdef ICONV_DEBUG
 -#define ICDEBUG(format, ...) printf("%s: "format, __func__ , __VA_ARGS__)
 +#define ICDEBUG(format, ...) printf("%s: "format, __func__ , ## __VA_ARGS__)
  #else
  #define ICDEBUG(format, ...)
  #endif
 _______________________________________________
 svn-src-all@freebsd.org mailing list
 http://lists.freebsd.org/mailman/listinfo/svn-src-all
 To unsubscribe, send any mail to "svn-src-all-unsubscribe@freebsd.org"
 



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