diff options
| author | Ed Tanous <edtanous@google.com> | 2022-01-25 21:07:45 +0300 |
|---|---|---|
| committer | Ed Tanous <ed@tanous.net> | 2022-02-07 19:41:37 +0300 |
| commit | dcf2ebc020257bc298d948fcb4da761c284e84d7 (patch) | |
| tree | 5e1ee6dc29480f7b26cd39bf9a9bd0d6759bfdaa /redfish-core/lib/ethernet.hpp | |
| parent | 104f09c95af2b3068e7afd655e4cebcad2bf0617 (diff) | |
| download | bmcweb-dcf2ebc020257bc298d948fcb4da761c284e84d7.tar.xz | |
Enable readability-redundant-control-flow checks
These checks are a nice addition to our static analysis, as they
simplify code quite a bit, as can be seen by this diff being negative
lines.
Signed-off-by: Ed Tanous <edtanous@google.com>
Change-Id: I60ede4ad23d7e5337e811d70ddcab24bf8986891
Diffstat (limited to 'redfish-core/lib/ethernet.hpp')
| -rw-r--r-- | redfish-core/lib/ethernet.hpp | 6 |
1 files changed, 1 insertions, 5 deletions
diff --git a/redfish-core/lib/ethernet.hpp b/redfish-core/lib/ethernet.hpp index 7c2da78843..e4e5eb9d62 100644 --- a/redfish-core/lib/ethernet.hpp +++ b/redfish-core/lib/ethernet.hpp @@ -1853,11 +1853,7 @@ inline void parseInterfaceData(nlohmann::json& jsonResponse, inline bool verifyNames(const std::string& parent, const std::string& iface) { - if (!boost::starts_with(iface, parent + "_")) - { - return false; - } - return true; + return boost::starts_with(iface, parent + "_"); } inline void requestEthernetInterfacesRoutes(App& app) |