From: Anders Kaseorg Date: Sat, 5 Nov 2016 00:38:08 +0000 (-0400) Subject: src/rx/rx_packet.c: Fix misleading indentation X-Git-Tag: upstream/1.8.0_pre1^2~28 X-Git-Url: https://git.michaelhowe.org/gitweb/?a=commitdiff_plain;h=0aeb8c17a2701169ddb7397d951c73cf361087c8;p=packages%2Fo%2Fopenafs.git src/rx/rx_packet.c: Fix misleading indentation Fixes these warnings (errors with --enable-checking) from GCC 6.2: rx_packet.c: In function ‘rxi_ReceiveDebugPacket’: rx_packet.c:2009:9: error: this ‘if’ clause does not guard... [-Werror=misleading-indentation] if (rx_stats_active) ^~ rx_packet.c:2011:6: note: ...this statement, but the latter is misleadingly indented as if it is guarded by the ‘if’ s = (afs_int32 *) & rx_stats; ^ rx_packet.c:2017:9: error: this ‘if’ clause does not guard... [-Werror=misleading-indentation] if (rx_stats_active) ^~ rx_packet.c:2019:6: note: ...this statement, but the latter is misleadingly indented as if it is guarded by the ‘if’ rxi_SendDebugPacket(ap, asocket, ahost, aport, istack); ^~~~~~~~~~~~~~~~~~~ Change-Id: Iaecedf63e9ed393607b8700b892aea7678c774b3 Reviewed-on: https://gerrit.openafs.org/12436 Reviewed-by: Benjamin Kaduk Tested-by: BuildBot --- diff --git a/src/rx/rx_packet.c b/src/rx/rx_packet.c index 2c5103689..37f995746 100644 --- a/src/rx/rx_packet.c +++ b/src/rx/rx_packet.c @@ -2006,16 +2006,16 @@ rxi_ReceiveDebugPacket(struct rx_packet *ap, osi_socket asocket, return ap; /* Since its all int32s convert to network order with a loop. */ - if (rx_stats_active) - MUTEX_ENTER(&rx_stats_mutex); + if (rx_stats_active) + MUTEX_ENTER(&rx_stats_mutex); s = (afs_int32 *) & rx_stats; for (i = 0; i < sizeof(rx_stats) / sizeof(afs_int32); i++, s++) rx_PutInt32(ap, i * sizeof(afs_int32), htonl(*s)); tl = ap->length; ap->length = sizeof(rx_stats); - if (rx_stats_active) - MUTEX_EXIT(&rx_stats_mutex); + if (rx_stats_active) + MUTEX_EXIT(&rx_stats_mutex); rxi_SendDebugPacket(ap, asocket, ahost, aport, istack); ap->length = tl; break;