cmds: Change cmd_create_putval() to accept a strbuf_t.

diff --git a/src/utils/cmds/putval.c b/src/utils/cmds/putval.c
index 616b6a4bd7fea601e447f6a07e56ce5a100e8554..a27e6aec4379853f38b642418ffcd13e45170b80 100644 (file)
--- a/src/utils/cmds/putval.c
+++ b/src/utils/cmds/putval.c
@@ -242,9 +242,8 @@ cmd_status_t cmd_handle_putval(FILE *fh, char *buffer) {
  *
  *   PUTVAL metric_name label:key="value" interval=10.000 42
  */
-int cmd_create_putval(char *ret, size_t ret_len, /* {{{ */
-                      metric_t const *m) {
-  if ((ret == NULL) || (ret_len == 0) || (m == NULL)) {
+int cmd_create_putval(strbuf_t *buf, metric_t const *m) { /* {{{ */
+  if ((buf == NULL) || (m == NULL)) {
     return EINVAL;
   }
 
@@ -254,10 +253,9 @@ int cmd_create_putval(char *ret, size_t ret_len, /* {{{ */
     return status;
   }
 
-  strbuf_t buf = STRBUF_CREATE_FIXED(ret, ret_len);
-  strbuf_print(&buf, "PUTVAL \"");
-  strbuf_print_escaped(&buf, id_buf.ptr, "\\\"\n\r\t", '\\');
-  strbuf_printf(&buf, "\" interval=%.3f", CDTIME_T_TO_DOUBLE(m->interval));
+  strbuf_print(buf, "PUTVAL \"");
+  strbuf_print_escaped(buf, id_buf.ptr, "\\\"\n\r\t", '\\');
+  strbuf_printf(buf, "\" interval=%.3f", CDTIME_T_TO_DOUBLE(m->interval));
   /* TODO(octo): print option to set the value type. */
-  return value_marshal_text(&buf, m->value, m->family->type);
+  return value_marshal_text(buf, m->value, m->family->type);
 } /* }}} int cmd_create_putval */

diff --git a/src/utils/cmds/putval.h b/src/utils/cmds/putval.h
index e535e312932610fe197c0494e45a52197f54b4ec..9985d3e0516759052a41c6d16aa910d6889e9324 100644 (file)
--- a/src/utils/cmds/putval.h
+++ b/src/utils/cmds/putval.h
@@ -41,6 +41,6 @@ cmd_status_t cmd_handle_putval(FILE *fh, char *buffer);
 
 void cmd_destroy_putval(cmd_putval_t *putval);
 
-int cmd_create_putval(char *ret, size_t ret_len, metric_t const *m);
+int cmd_create_putval(strbuf_t *buf, metric_t const *m);
 
 #endif /* UTILS_CMD_PUTVAL_H */