projects
/
collectd.git
/ blobdiff
commit
grep
author
committer
pickaxe
?
search:
re
summary
|
shortlog
|
log
|
commit
|
commitdiff
|
tree
raw
|
inline
| side by side
amqp, write_graphite and write_kafka plugins: Implement the "[Graphite]PreserveSepara...
[collectd.git]
/
src
/
threshold.c
diff --git
a/src/threshold.c
b/src/threshold.c
index
94a4f02
..
1d9bcf9
100644
(file)
--- a/
src/threshold.c
+++ b/
src/threshold.c
@@
-24,14
+24,13
@@
**/
#include "collectd.h"
**/
#include "collectd.h"
+
#include "common.h"
#include "plugin.h"
#include "utils_avltree.h"
#include "utils_cache.h"
#include "utils_threshold.h"
#include "common.h"
#include "plugin.h"
#include "utils_avltree.h"
#include "utils_cache.h"
#include "utils_threshold.h"
-#include <assert.h>
-
/*
* Threshold management
* ====================
/*
* Threshold management
* ====================
@@
-217,7
+216,6
@@
static int ut_config_type_hysteresis (threshold_t *th, oconfig_item_t *ci)
static int ut_config_type (const threshold_t *th_orig, oconfig_item_t *ci)
{
static int ut_config_type (const threshold_t *th_orig, oconfig_item_t *ci)
{
- int i;
threshold_t th;
int status = 0;
threshold_t th;
int status = 0;
@@
-246,7
+244,7
@@
static int ut_config_type (const threshold_t *th_orig, oconfig_item_t *ci)
th.hysteresis = 0;
th.flags = UT_FLAG_INTERESTING; /* interesting by default */
th.hysteresis = 0;
th.flags = UT_FLAG_INTERESTING; /* interesting by default */
- for (i = 0; i < ci->children_num; i++)
+ for (i
nt i
= 0; i < ci->children_num; i++)
{
oconfig_item_t *option = ci->children + i;
{
oconfig_item_t *option = ci->children + i;
@@
-311,7
+309,6
@@
static int ut_config_plugin_instance (threshold_t *th, oconfig_item_t *ci)
static int ut_config_plugin (const threshold_t *th_orig, oconfig_item_t *ci)
{
static int ut_config_plugin (const threshold_t *th_orig, oconfig_item_t *ci)
{
- int i;
threshold_t th;
int status = 0;
threshold_t th;
int status = 0;
@@
-333,7
+330,7
@@
static int ut_config_plugin (const threshold_t *th_orig, oconfig_item_t *ci)
memcpy (&th, th_orig, sizeof (th));
sstrncpy (th.plugin, ci->values[0].value.string, sizeof (th.plugin));
memcpy (&th, th_orig, sizeof (th));
sstrncpy (th.plugin, ci->values[0].value.string, sizeof (th.plugin));
- for (i = 0; i < ci->children_num; i++)
+ for (i
nt i
= 0; i < ci->children_num; i++)
{
oconfig_item_t *option = ci->children + i;
{
oconfig_item_t *option = ci->children + i;
@@
-357,7
+354,6
@@
static int ut_config_plugin (const threshold_t *th_orig, oconfig_item_t *ci)
static int ut_config_host (const threshold_t *th_orig, oconfig_item_t *ci)
{
static int ut_config_host (const threshold_t *th_orig, oconfig_item_t *ci)
{
- int i;
threshold_t th;
int status = 0;
threshold_t th;
int status = 0;
@@
-379,7
+375,7
@@
static int ut_config_host (const threshold_t *th_orig, oconfig_item_t *ci)
memcpy (&th, th_orig, sizeof (th));
sstrncpy (th.host, ci->values[0].value.string, sizeof (th.host));
memcpy (&th, th_orig, sizeof (th));
sstrncpy (th.host, ci->values[0].value.string, sizeof (th.host));
- for (i = 0; i < ci->children_num; i++)
+ for (i
nt i
= 0; i < ci->children_num; i++)
{
oconfig_item_t *option = ci->children + i;
{
oconfig_item_t *option = ci->children + i;
@@
-551,10
+547,9
@@
static int ut_report_state (const data_set_t *ds,
{
gauge_t value;
gauge_t sum;
{
gauge_t value;
gauge_t sum;
- size_t i;
sum = 0.0;
sum = 0.0;
- for (i = 0; i < vl->values_len; i++)
+ for (
size_t
i = 0; i < vl->values_len; i++)
{
if (isnan (values[i]))
continue;
{
if (isnan (values[i]))
continue;
@@
-699,7
+694,6
@@
static int ut_check_one_threshold (const data_set_t *ds,
{ /* {{{ */
int ret = -1;
int ds_index = -1;
{ /* {{{ */
int ret = -1;
int ds_index = -1;
- size_t i;
gauge_t values_copy[ds->ds_num];
memcpy (values_copy, values, sizeof (values_copy));
gauge_t values_copy[ds->ds_num];
memcpy (values_copy, values, sizeof (values_copy));
@@
-718,7
+712,7
@@
static int ut_check_one_threshold (const data_set_t *ds,
}
/* Prepare `sum' and `num'. */
}
/* Prepare `sum' and `num'. */
- for (i = 0; i < ds->ds_num; i++)
+ for (
size_t
i = 0; i < ds->ds_num; i++)
if (!isnan (values[i]))
{
num++;
if (!isnan (values[i]))
{
num++;
@@
-728,17
+722,17
@@
static int ut_check_one_threshold (const data_set_t *ds,
if ((num == 0) /* All data sources are undefined. */
|| (sum == 0.0)) /* Sum is zero, cannot calculate percentage. */
{
if ((num == 0) /* All data sources are undefined. */
|| (sum == 0.0)) /* Sum is zero, cannot calculate percentage. */
{
- for (i = 0; i < ds->ds_num; i++)
+ for (
size_t
i = 0; i < ds->ds_num; i++)
values_copy[i] = NAN;
}
else /* We can actually calculate the percentage. */
{
values_copy[i] = NAN;
}
else /* We can actually calculate the percentage. */
{
- for (i = 0; i < ds->ds_num; i++)
+ for (
size_t
i = 0; i < ds->ds_num; i++)
values_copy[i] = 100.0 * values[i] / sum;
}
} /* if (UT_FLAG_PERCENTAGE) */
values_copy[i] = 100.0 * values[i] / sum;
}
} /* if (UT_FLAG_PERCENTAGE) */
- for (i = 0; i < ds->ds_num; i++)
+ for (
size_t
i = 0; i < ds->ds_num; i++)
{
int status;
{
int status;
@@
-868,15
+862,12
@@
static int ut_missing (const value_list_t *vl,
static int ut_config (oconfig_item_t *ci)
{ /* {{{ */
static int ut_config (oconfig_item_t *ci)
{ /* {{{ */
- int i;
int status = 0;
int old_size = c_avl_size (threshold_tree);
int status = 0;
int old_size = c_avl_size (threshold_tree);
- threshold_t th;
-
if (threshold_tree == NULL)
{
if (threshold_tree == NULL)
{
- threshold_tree = c_avl_create ((
void *
) strcmp);
+ threshold_tree = c_avl_create ((
int (*) (const void *, const void *)
) strcmp);
if (threshold_tree == NULL)
{
ERROR ("ut_config: c_avl_create failed.");
if (threshold_tree == NULL)
{
ERROR ("ut_config: c_avl_create failed.");
@@
-884,17
+875,15
@@
static int ut_config (oconfig_item_t *ci)
}
}
}
}
- memset (&th, '\0', sizeof (th));
- th.warning_min = NAN;
- th.warning_max = NAN;
- th.failure_min = NAN;
- th.failure_max = NAN;
-
- th.hits = 0;
- th.hysteresis = 0;
- th.flags = UT_FLAG_INTERESTING; /* interesting by default */
+ threshold_t th = {
+ .warning_min = NAN,
+ .warning_max = NAN,
+ .failure_min = NAN,
+ .failure_max = NAN,
+ .flags = UT_FLAG_INTERESTING /* interesting by default */
+ };
- for (i = 0; i < ci->children_num; i++)
+ for (i
nt i
= 0; i < ci->children_num; i++)
{
oconfig_item_t *option = ci->children + i;
{
oconfig_item_t *option = ci->children + i;