Add std::string to RMI with cleanups.

Signed-off-by: Samuel K. Gutierrez <[email protected]>

Author: samuelkgutierrez

src/qvi-bbuff-rmi.h

@@ -1,6 +1,6 @@
 /* -*- Mode: C++; c-basic-offset:4; indent-tabs-mode:nil -*- */
 /*
- * Copyright (c) 2021-2023 Triad National Security, LLC
+ * Copyright (c) 2021-2024 Triad National Security, LLC
  *                         All rights reserved.
  *
  * This file is part of the quo-vadis project. See the LICENSE file at the
@@ -19,6 +19,7 @@
  * h = qvi_line_hwpool_t *
  * i = int
  * s = char *
+ * s = std::string
  * t = qvi_task_id_t
  * z = qvi_bbuff_rmi_zero_msg_t
  */
@@ -30,10 +31,8 @@
 #include "qvi-hwloc.h"
 #include "qvi-hwpool.h"
 
-#include <string>
-
 // 'Null' cpuset representation as a string.
-#define QV_BUFF_RMI_NULL_CPUSET ""
+static const cstr_t QV_BUFF_RMI_NULL_CPUSET = "";
 
 // Byte containers
 using qvi_bbuff_rmi_bytes_in_t = std::pair<void *, size_t>;
@@ -227,6 +226,15 @@ qvi_bbuff_rmi_pack_type_picture(
     picture += "s";
 }
 
+template<>
+inline void
+qvi_bbuff_rmi_pack_type_picture(
+    std::string &picture,
+    std::string &
+) {
+    picture += "s";
+}
+
 template<>
 inline void
 qvi_bbuff_rmi_pack_type_picture(
@@ -361,6 +369,17 @@ qvi_bbuff_rmi_pack_item_impl(
     return qvi_bbuff_append(buff, data, strlen(data) + 1);
 }
 
+/**
+ * Packs std::string
+ */
+inline int
+qvi_bbuff_rmi_pack_item(
+    qvi_bbuff_t *buff,
+    const std::string &data
+) {
+    return qvi_bbuff_rmi_pack_item_impl(buff, data.c_str());
+}
+
 /**
  * Packs const char *
  */
@@ -798,7 +817,6 @@ qvi_bbuff_rmi_unpack_item(
     );
     if (rc != QV_SUCCESS) goto out;
     total_bw += bw;
-    buffpos += bw;
 out:
     if (rc != QV_SUCCESS) {
         total_bw = 0;
@@ -901,7 +919,6 @@ qvi_bbuff_rmi_unpack_item(
     );
     if (rc != QV_SUCCESS) goto out;
     total_bw += bw;
-    buffpos += bw;
 out:
     if (rc != QV_SUCCESS) {
         total_bw = 0;