[Xorp-cvs] XORP cvs commit: xorp/libxorp

Pavlin Radoslavov pavlin at icir.org
Thu Oct 23 17:58:59 PDT 2008


CVSROOT:	/usr/local/www/data/cvs
Module name:	xorp
Changes by:	pavlin at chum.icir.org	2008-10-24 00:58:59 UTC

XORP CVS repository


Modified files:
	libxorp       xlog.h 

Log message:
	Add-back the "do { ... } while (0)" construct in the XLOG_FN()
	macro definition that was removed in rev. 1.21 of this file,
	because it is required.
	
	It is needed in places like the following:
	
	    if (foo)
	        XLOG_INFO("Yes");
	    else
	        XLOG_INFO("No");
	
	The construct in rev. 1.21 was just "{ ... }", hence the above
	code was expanded into:
	
	    if (foo)
	    {
	        ...
	    };
	    else
	    {
	        ...
	    };
	
	This code will fail to compile because of the semicolon before the
	"else" statement.

Revision  Changes                              Path
1.23      +3 -3;  commitid: c86249011afd41a7;  xorp/libxorp/xlog.h



More information about the Xorp-cvs mailing list