diff options
| author | David Gibson <david@gibson.dropbear.id.au> | 2023-09-29 15:50:20 +1000 |
|---|---|---|
| committer | Stefano Brivio <sbrivio@redhat.com> | 2023-10-04 23:23:56 +0200 |
| commit | c0efa4e97f20848570552286403574bf228b6420 (patch) | |
| tree | fd0bc3699dd700673ec35123703ee7c160ce093d /util.c | |
| parent | 6471c7d01b0aad9d144448290557fcd783562228 (diff) | |
| download | passt-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 'util.c')
0 files changed, 0 insertions, 0 deletions