MINOR: stats: make stats_dump_fields_json() directly take flags
It used to take an inverted flag for STAT_STARTED, let's make it take
the raw flags instead.
diff --git a/src/stats.c b/src/stats.c
index d464bd5..7197112 100644
--- a/src/stats.c
+++ b/src/stats.c
@@ -603,12 +603,12 @@
/* Dump all fields from <stats> into <out> using a typed "field:desc:type:value" format */
static int stats_dump_fields_json(struct buffer *out,
const struct field *stats,
- int first_stat)
+ unsigned int flags)
{
int field;
int started = 0;
- if (!first_stat && !chunk_strcat(out, ","))
+ if ((flags & STAT_STARTED) && !chunk_strcat(out, ","))
return 0;
if (!chunk_strcat(out, "["))
return 0;
@@ -663,8 +663,8 @@
err:
chunk_reset(out);
- if (!first_stat)
- chunk_strcat(out, ",");
+ if (flags & STAT_STARTED)
+ chunk_strcat(out, ",");
chunk_appendf(out, "{\"errorStr\":\"output buffer too short\"}");
return 0;
}
@@ -1362,9 +1362,7 @@
else if (appctx->ctx.stats.flags & STAT_FMT_TYPED)
ret = stats_dump_fields_typed(&trash, stats);
else if (appctx->ctx.stats.flags & STAT_FMT_JSON)
- ret = stats_dump_fields_json(&trash, stats,
- !(appctx->ctx.stats.flags &
- STAT_STARTED));
+ ret = stats_dump_fields_json(&trash, stats, appctx->ctx.stats.flags);
else
ret = stats_dump_fields_csv(&trash, stats);