From 084c2c1687fda07c768837e8fd7cc0904818f977 Mon Sep 17 00:00:00 2001 From: Simon Wilkinson Date: Mon, 14 Nov 2011 10:28:38 +0000 Subject: [PATCH] rx: Remove trailing else from RX_KERNEL_TRACE The RX_KERNEL_TRACE ifdefs all contain a trailing else, which makes no logicial sense given the rest of the code. The only effect of this trailing else is to cause packets to not be sent when tracing is enabled, which probably isn't the desired behaviour. Change-Id: I2175b1d8270cd096361f349a467245b49efabf6b Reviewed-on: http://gerrit.openafs.org/7001 Reviewed-by: Derrick Brashear Reviewed-by: Jeffrey Altman Tested-by: BuildBot --- src/rx/rx_packet.c | 8 ++++---- 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/src/rx/rx_packet.c b/src/rx/rx_packet.c index 78d2e5e81f..a92ae38257 100644 --- a/src/rx/rx_packet.c +++ b/src/rx/rx_packet.c @@ -2138,7 +2138,7 @@ rxi_SendDebugPacket(struct rx_packet *apacket, osi_socket asocket, afs_Trace1(afs_iclSetp, CM_TRACE_TIMESTAMP, ICL_TYPE_STRING, "before osi_NetSend()"); AFS_GUNLOCK(); - } else + } #else if (waslocked) AFS_GUNLOCK(); @@ -2155,7 +2155,7 @@ rxi_SendDebugPacket(struct rx_packet *apacket, osi_socket asocket, "after osi_NetSend()"); if (!waslocked) AFS_GUNLOCK(); - } else + } #else if (waslocked) AFS_GLOCK(); @@ -2265,7 +2265,7 @@ rxi_SendPacket(struct rx_call *call, struct rx_connection *conn, afs_Trace1(afs_iclSetp, CM_TRACE_TIMESTAMP, ICL_TYPE_STRING, "before osi_NetSend()"); AFS_GUNLOCK(); - } else + } #else if (waslocked) AFS_GUNLOCK(); @@ -2305,7 +2305,7 @@ rxi_SendPacket(struct rx_call *call, struct rx_connection *conn, "after osi_NetSend()"); if (!waslocked) AFS_GUNLOCK(); - } else + } #else if (waslocked) AFS_GLOCK();