diff options
| author | Ed Tanous <edtanous@google.com> | 2022-01-04 23:47:01 +0300 |
|---|---|---|
| committer | Ed Tanous <ed@tanous.net> | 2022-01-05 23:07:08 +0300 |
| commit | 248d02303212eb1b7b9a81757489a23af36546a8 (patch) | |
| tree | ecae6fbf55b5e9d25268397b9ad41e377effab47 /redfish-core/include/event_service_manager.hpp | |
| parent | 1ec43ee36d52f415261b27653b1ba6311ebb5bfe (diff) | |
| download | bmcweb-248d02303212eb1b7b9a81757489a23af36546a8.tar.xz | |
Make code compile with clang
One minor shadowed variable that needed it's name changed, and a missing
inline statement.
Tested: Code now builds in clang-13
Signed-off-by: Ed Tanous <edtanous@google.com>
Change-Id: I83b662a2818e1469dfeb29b818338346e40cb832
Diffstat (limited to 'redfish-core/include/event_service_manager.hpp')
| -rw-r--r-- | redfish-core/include/event_service_manager.hpp | 6 |
1 files changed, 3 insertions, 3 deletions
diff --git a/redfish-core/include/event_service_manager.hpp b/redfish-core/include/event_service_manager.hpp index 9349cd84b0..642a3029b5 100644 --- a/redfish-core/include/event_service_manager.hpp +++ b/redfish-core/include/event_service_manager.hpp @@ -500,7 +500,7 @@ class Subscription : public persistent_data::UserSubscription } #endif - void filterAndSendReports(const std::string& id, + void filterAndSendReports(const std::string& reportId, const telemetry::TimestampReadings& var) { std::string mrdUri = telemetry::metricReportDefinitionUri + ("/" + id); @@ -517,11 +517,11 @@ class Subscription : public persistent_data::UserSubscription } nlohmann::json msg; - if (!telemetry::fillReport(msg, id, var)) + if (!telemetry::fillReport(msg, reportId, var)) { BMCWEB_LOG_ERROR << "Failed to fill the MetricReport for DBus " "Report with id " - << id; + << reportId; return; } |