[Pcsclite-cvs-commit] r6220 - /trunk/Drivers/ccid/src/ifdhandler.c

[rousseau at users.alioth.debian.org]

Author: rousseau
Date: Mon Feb 13 09:10:36 2012
New Revision: 6220

URL: http://svn.debian.org/wsvn/pcsclite/?sc=1&rev=6220
Log:
CreateChannelByNameOrChannel(): add explicit braces

Fix compiler warning:
ifdhandler.c: In function 'CreateChannelByNameOrChannel':
ifdhandler.c:81:5: warning: suggest explicit braces to avoid ambiguous 'else' [-Wparentheses]

The problem is that DEBUG_INFO3() is not a code block but a "dangerous"
if () macro.

Modified:
    trunk/Drivers/ccid/src/ifdhandler.c

Modified: trunk/Drivers/ccid/src/ifdhandler.c
URL: http://svn.debian.org/wsvn/pcsclite/trunk/Drivers/ccid/src/ifdhandler.c?rev=6220&op=diff
==============================================================================
--- trunk/Drivers/ccid/src/ifdhandler.c (original)
+++ trunk/Drivers/ccid/src/ifdhandler.c Mon Feb 13 09:10:36 2012
@@ -79,9 +79,13 @@
 		init_driver();
 
 	if (lpcDevice)
+	{
 		DEBUG_INFO3("lun: %lX, device: %s", Lun, lpcDevice);
+	}
 	else
+	{
 		DEBUG_INFO3("lun: %lX, Channel: %lX", Lun, Channel);
+	}
 
 	if (-1 == (reader_index = GetNewReaderIndex(Lun)))
 		return IFD_COMMUNICATION_ERROR;