diff options
| author | Donatas Abraitis <donatas.abraitis@gmail.com> | 2021-06-29 14:43:16 +0300 |
|---|---|---|
| committer | Donatas Abraitis <donatas.abraitis@gmail.com> | 2021-06-29 22:27:49 +0300 |
| commit | 0e2d70760d8422e38dd4d6f8ceed2b960b0ed433 (patch) | |
| tree | 4a4a7dfb21d429485b7245bc6a869b6c83bef152 /lib/ntop.c | |
| parent | 11dbcdd35ccf9b9978a61f6f501ab9062c9e5687 (diff) | |
lib: Avoid using assignments within checks
Signed-off-by: Donatas Abraitis <donatas.abraitis@gmail.com>
Diffstat (limited to 'lib/ntop.c')
| -rw-r--r-- | lib/ntop.c | 14 |
1 files changed, 9 insertions, 5 deletions
diff --git a/lib/ntop.c b/lib/ntop.c index ccbf8793d3..1b2dd7a6d1 100644 --- a/lib/ntop.c +++ b/lib/ntop.c @@ -40,14 +40,18 @@ static inline void putbyte(uint8_t bytex, char **posx) bool zero = false; int byte = bytex, tmp, a, b; - if ((tmp = byte - 200) >= 0) { + tmp = byte - 200; + if (tmp >= 0) { *pos++ = '2'; zero = true; byte = tmp; - } else if ((tmp = byte - 100) >= 0) { - *pos++ = '1'; - zero = true; - byte = tmp; + } else { + tmp = byte - 100; + if (tmp >= 0) { + *pos++ = '1'; + zero = true; + byte = tmp; + } } /* make sure the compiler knows the value range of "byte" */ |
