diff options
author | David Gibson <david@gibson.dropbear.id.au> | 2022-09-28 14:33:20 +1000 |
---|---|---|
committer | Stefano Brivio <sbrivio@redhat.com> | 2022-09-29 12:22:05 +0200 |
commit | ab96da98cd6ccabf184c12575ab5116de51c75c3 (patch) | |
tree | 286487d09ff1bf04c8d8bb5610a43bfcdad9cee2 /tcp.c | |
parent | eb5e1230386ce2eafa2ef1770bb5ffd3e918fdc6 (diff) | |
download | passt-ab96da98cd6ccabf184c12575ab5116de51c75c3.tar passt-ab96da98cd6ccabf184c12575ab5116de51c75c3.tar.gz passt-ab96da98cd6ccabf184c12575ab5116de51c75c3.tar.bz2 passt-ab96da98cd6ccabf184c12575ab5116de51c75c3.tar.lz passt-ab96da98cd6ccabf184c12575ab5116de51c75c3.tar.xz passt-ab96da98cd6ccabf184c12575ab5116de51c75c3.tar.zst passt-ab96da98cd6ccabf184c12575ab5116de51c75c3.zip |
Don't shadow 'i' in conf_ports()
The counter 'i' is used in a number of places in conf_ports(), but in one
of those we unnecessarily shadow it in an inner scope. We could re-use the
same 'i' every time, but each use is logically separate, so instead remove
the outer declaration and declare it locally in each of the clauses where
we need it.
While we're there change it from a signed to unsigned int, since it's used
to iterate over port numbers which are generally treated as unsigned.
Signed-off-by: David Gibson <david@gibson.dropbear.id.au>
Signed-off-by: Stefano Brivio <sbrivio@redhat.com>
Diffstat (limited to 'tcp.c')
0 files changed, 0 insertions, 0 deletions