diff options
author | Ed Tanous <edtanous@google.com> | 2022-01-25 20:49:11 +0300 |
---|---|---|
committer | Ed Tanous <ed@tanous.net> | 2022-01-28 22:43:59 +0300 |
commit | 26f6976f3905d7b20690bbf794fba746c1929ed3 (patch) | |
tree | 7cfaea46d4e70ea7b46275195f7d7024cb999dac /redfish-core/lib/log_services.hpp | |
parent | bb05f2227d53d590534d1df6e6a622e13cea76ae (diff) | |
download | bmcweb-26f6976f3905d7b20690bbf794fba746c1929ed3.tar.xz |
Enable readability-container-size-empty tests
This one is a little trivial, but it does help in readability.
Signed-off-by: Ed Tanous <edtanous@google.com>
Change-Id: I5366d4eec8af2f781b3bad804131ae2eb806e3aa
Diffstat (limited to 'redfish-core/lib/log_services.hpp')
-rw-r--r-- | redfish-core/lib/log_services.hpp | 6 |
1 files changed, 3 insertions, 3 deletions
diff --git a/redfish-core/lib/log_services.hpp b/redfish-core/lib/log_services.hpp index 3877541338..41438cb796 100644 --- a/redfish-core/lib/log_services.hpp +++ b/redfish-core/lib/log_services.hpp @@ -1115,7 +1115,7 @@ static int fillEventLogEntryJson(const std::string& logEntryID, boost::split(logEntryFields, entry, boost::is_any_of(","), boost::token_compress_on); // We need at least a MessageId to be valid - if (logEntryFields.size() < 1) + if (logEntryFields.empty()) { return 1; } @@ -1971,12 +1971,12 @@ inline void requestRoutesSystemHostLoggerCollection(App& app) } // If vector is empty, that means skip value larger than total // log count - if (logEntries.size() == 0) + if (logEntries.empty()) { asyncResp->res.jsonValue["Members@odata.count"] = logCount; return; } - if (logEntries.size() > 0) + if (!logEntries.empty()) { for (size_t i = 0; i < logEntries.size(); i++) { |