summaryrefslogtreecommitdiffstats
path: root/include
diff options
context:
space:
mode:
authorMatt Spinler <spinler@us.ibm.com>2019-01-11 15:37:06 -0600
committerEd Tanous <ed.tanous@intel.com>2019-01-12 01:44:34 +0000
commit4ae611d9de1504b68f521e9837ddb97e0dc89d27 (patch)
tree8fdff4a042fce57781e1c0cfeac4cb23f3aeec28 /include
parent8f626357d5ebee1bf8039ad6e85f5e02c64c9076 (diff)
downloadbmcweb-4ae611d9de1504b68f521e9837ddb97e0dc89d27.tar.gz
bmcweb-4ae611d9de1504b68f521e9837ddb97e0dc89d27.zip
REST: Add missing return statement to dbusArgSplit
The function dbusArgSplit() was missing the return statement at the end, which was causing a warning in the bitbake compile logs. The function apparently still worked fine without it, but one wouldn't think it would just by looking at it. Change-Id: Ifcd1f742de995eb19685d0ef37036860ebbf8b95 Signed-off-by: Matt Spinler <spinler@us.ibm.com>
Diffstat (limited to 'include')
-rw-r--r--include/openbmc_dbus_rest.hpp2
1 files changed, 2 insertions, 0 deletions
diff --git a/include/openbmc_dbus_rest.hpp b/include/openbmc_dbus_rest.hpp
index da59d25..fb23a8b 100644
--- a/include/openbmc_dbus_rest.hpp
+++ b/include/openbmc_dbus_rest.hpp
@@ -467,6 +467,8 @@ std::vector<std::string> dbusArgSplit(const std::string &string)
break;
}
}
+
+ return ret;
}
int convertJsonToDbus(sd_bus_message *m, const std::string &arg_type,
OpenPOWER on IntegriCloud