diff options
| author | Alexis La Goutte <alexis.lagoutte@gmail.com> | 2015-12-01 15:09:27 +0100 |
|---|---|---|
| committer | Anders Broman <a.broman58@gmail.com> | 2015-12-01 21:08:04 +0000 |
| commit | 21ac94a323e6ede4407dddf1190f5bfe5f7915d0 (patch) | |
| tree | 6b87be33a8d9dc2bd10b11ca59542ac1ce00b6dd /epan/stats_tree.c | |
| parent | f00af593b914534c88ce44af6bd31c6a9fc3d43d (diff) | |
stats_tree: re-indent else for avoid warning found by PVS Studio (V705)
It is possible that 'else' block was forgotten or commented out, thus altering the program's operation logics
Change-Id: Ie246592282bd0791021159f90b4b8aeb92cff86f
Reviewed-on: https://code.wireshark.org/review/12333
Reviewed-by: Anders Broman <a.broman58@gmail.com>
Diffstat (limited to 'epan/stats_tree.c')
| -rw-r--r-- | epan/stats_tree.c | 16 |
1 files changed, 9 insertions, 7 deletions
diff --git a/epan/stats_tree.c b/epan/stats_tree.c index 59c9b0c6b5..c837277f96 100644 --- a/epan/stats_tree.c +++ b/epan/stats_tree.c @@ -725,16 +725,18 @@ get_range(char *rngstr) /* string == "X-?" */ if (*(split[0]) != '\0') { rng->floor = (gint)strtol(split[0],NULL,10); - } else - /* string == "-?" */ - rng->floor = G_MININT; + } else { + /* string == "-?" */ + rng->floor = G_MININT; + } /* string != "?-" */ - if (*(split[1]) != '\0') { + if (*(split[1]) != '\0') { rng->ceil = (gint)strtol(split[1],NULL,10); - } else - /* string == "?-" */ - rng->ceil = G_MAXINT; + } else { + /* string == "?-" */ + rng->ceil = G_MAXINT; + } } g_strfreev(split); |