rx: remove trailing semicolons from FBSD mutex operations 53/12853/2
authorBenjamin Kaduk <kaduk@mit.edu>
Fri, 5 Jan 2018 04:00:15 +0000 (22:00 -0600)
committerBenjamin Kaduk <kaduk@mit.edu>
Tue, 9 Jan 2018 02:46:25 +0000 (21:46 -0500)
commit0760feb7992e1e39f716c5f583fe7f6e85584262
tree921e8632fd914bcf3d6a2d6aca2b77473526a828
parentdecb4308d4e18ad9f6f181e3df5f737698dba7ad
rx: remove trailing semicolons from FBSD mutex operations

Since the first introduction of FreeBSD support, the macros
(MUTEX_ENTER, etc.) for kernel mutex operations have included
trailing semicolons, unique among all the platforms.

This did not cause problems until the recent work on rx event
handlers, which put a MUTEX_ENTER() in the body of an 'if' clause
with no brackets, and attempted to follow it with an 'else' clause.
This results in the following (rather obtuse) compiler error:

   /root/openafs/src/rx/rx.c:3666:5: error: expected expression
       else
       ^

Which is more visible in the preprocessed source, as

   if (condition)
       expression;;
   else
       other_expression;

is clearly invalid C.

To fix the FreeBSD kernel module build, remove the unneeded semicolons.

Change-Id: I191009ad412852dcc03cd71a0982fe41a953301d
Reviewed-on: https://gerrit.openafs.org/12853
Tested-by: BuildBot <buildbot@rampaginggeek.com>
Reviewed-by: Benjamin Kaduk <kaduk@mit.edu>
src/rx/FBSD/rx_kmutex.h