diff options
| author | Vsevolod Stakhov <vsevolod@rspamd.com> | 2023-01-23 21:43:01 +0000 |
|---|---|---|
| committer | Vsevolod Stakhov <vsevolod@rspamd.com> | 2023-01-23 21:43:01 +0000 |
| commit | c978a76a0fbaed446b672be38596785f4bc468e9 (patch) | |
| tree | 3778ad1d8e2de6cb33ac374d957574ae8521fe8f /src/client | |
| parent | 5aaa3869e1637921dfd1196faad2b128dec97caf (diff) | |
| download | rspamd-c978a76a0fbaed446b672be38596785f4bc468e9.tar.gz rspamd-c978a76a0fbaed446b672be38596785f4bc468e9.zip | |
[Minor] Some style neats
Diffstat (limited to 'src/client')
| -rw-r--r-- | src/client/rspamc.cxx | 14 |
1 files changed, 5 insertions, 9 deletions
diff --git a/src/client/rspamc.cxx b/src/client/rspamc.cxx index 9fa56decd..2adc80a7c 100644 --- a/src/client/rspamc.cxx +++ b/src/client/rspamc.cxx @@ -938,7 +938,7 @@ static void rspamc_metric_output(FILE *out, const ucl_object_t *obj) { int got_scores = 0; - bool is_spam = false, is_skipped = false; + bool is_spam = false; double score = 0, required_score = 0, greylist_score =0, addheader_score = 0; auto print_protocol_string = [&](const char *ucl_name, const char *output_message) { @@ -1054,18 +1054,14 @@ rspamc_metric_output(FILE *out, const ucl_object_t *obj) } if (humanreport) { - /* XXX: why checkv2 does not provide "is_spam"? */ - elt = ucl_object_lookup(obj, "is_spam"); - if (elt) { - is_spam = ucl_object_toboolean(elt); - } + auto is_skipped = 0; elt = ucl_object_lookup(obj, "is_skipped"); - if (elt) { - is_skipped = ucl_object_toboolean(elt); + if (elt && ucl_object_toboolean(elt)) { + is_skipped = 1; } - fmt::print(out, ",spam={},skipped={}\n", is_spam ? 1 : 0, is_skipped ? 1 : 0); + fmt::print(out, ",spam={},skipped={}\n", is_spam ? 1 : 0, is_skipped); } else if (got_scores == 2) { fmt::print(out, |