aboutgitcodebugslistschat
path: root/contrib
diff options
context:
space:
mode:
authorDavid Gibson <david@gibson.dropbear.id.au>2023-09-29 15:50:20 +1000
committerStefano Brivio <sbrivio@redhat.com>2023-10-04 23:23:56 +0200
commitc0efa4e97f20848570552286403574bf228b6420 (patch)
treefd0bc3699dd700673ec35123703ee7c160ce093d /contrib
parent6471c7d01b0aad9d144448290557fcd783562228 (diff)
downloadpasst-c0efa4e97f20848570552286403574bf228b6420.tar
passt-c0efa4e97f20848570552286403574bf228b6420.tar.gz
passt-c0efa4e97f20848570552286403574bf228b6420.tar.bz2
passt-c0efa4e97f20848570552286403574bf228b6420.tar.lz
passt-c0efa4e97f20848570552286403574bf228b6420.tar.xz
passt-c0efa4e97f20848570552286403574bf228b6420.tar.zst
passt-c0efa4e97f20848570552286403574bf228b6420.zip
conf: Remove overly cryptic selection of forward table
In a couple of places in conf(), we use a local 'fwd' variable to reference one of our forwarding tables. The value depends on which command line option we're currently looking at, and is initialized rather cryptically from an assignment side-effect within the if condition checking that option. Newer versions of cppcheck complain about this assignment being an always true condition, but in any case it's both clearer and slightly shorter to use separate if branches for the two cases and set the forwarding parameter more directly. Signed-off-by: David Gibson <david@gibson.dropbear.id.au> Signed-off-by: Stefano Brivio <sbrivio@redhat.com>
Diffstat (limited to 'contrib')
0 files changed, 0 insertions, 0 deletions