diff options
| author | Ed Tanous <ed@tanous.net> | 2020-09-29 04:41:25 +0300 |
|---|---|---|
| committer | Ed Tanous <ed@tanous.net> | 2020-09-30 02:20:36 +0300 |
| commit | 4e08751b5debb2451ced16ee10dfed2c863edaa7 (patch) | |
| tree | ca5cc25b1a55b4ba012669049ffcde58d16c1f82 /include/ibm/locks.hpp | |
| parent | e7d1a1cfd59ed69cd11d1db76e2ae4a9a8771407 (diff) | |
| download | bmcweb-4e08751b5debb2451ced16ee10dfed2c863edaa7.tar.xz | |
modernize ibm management console
There was some modernization problems in the IBM console. These are all
minor, and unlikely to cause problems. The issues were:
1. Trivial destructors need to use the = default syntax
2. Several loops can be simplified into range based for loops
3. push_back should not be paired with make_pair. emplace_back should
be used instead.
Change-Id: I71b1d5437249d896a6f95c211e176deb676f985d
Diffstat (limited to 'include/ibm/locks.hpp')
| -rw-r--r-- | include/ibm/locks.hpp | 5 |
1 files changed, 2 insertions, 3 deletions
diff --git a/include/ibm/locks.hpp b/include/ibm/locks.hpp index 162abeeefa..e986f32f1e 100644 --- a/include/ibm/locks.hpp +++ b/include/ibm/locks.hpp @@ -188,8 +188,7 @@ class Lock return lockObject; } - virtual ~Lock() - {} + virtual ~Lock() = default; }; inline bool Lock::createPersistentLockFilePath() @@ -293,7 +292,7 @@ inline RcGetLockList Lock::getLockList(const ListOfSessionIds& listSessionId) // Push the whole lock record into a vector for returning // the json - lockList.push_back(std::make_pair(it->first, it->second)); + lockList.emplace_back(it->first, it->second); } // Go to next entry in map it++; |