Use std::move in more places
Plus various code simplifications
Change-Id: I19805e4a635e4c74afaff68f9d8968475217ec6e
diff --git a/src/security/pib/identity-container.cpp b/src/security/pib/identity-container.cpp
index 7c018ce..0e4749f 100644
--- a/src/security/pib/identity-container.cpp
+++ b/src/security/pib/identity-container.cpp
@@ -1,6 +1,6 @@
/* -*- Mode:C++; c-file-style:"gnu"; indent-tabs-mode:nil; -*- */
-/**
- * Copyright (c) 2013-2017 Regents of the University of California.
+/*
+ * Copyright (c) 2013-2018 Regents of the University of California.
*
* This file is part of ndn-cxx library (NDN C++ library with eXperimental eXtensions).
*
@@ -81,10 +81,10 @@
}
IdentityContainer::IdentityContainer(shared_ptr<PibImpl> pibImpl)
- : m_pibImpl(pibImpl)
+ : m_pibImpl(std::move(pibImpl))
{
- BOOST_ASSERT(pibImpl != nullptr);
- m_identityNames = pibImpl->getIdentities();
+ BOOST_ASSERT(m_pibImpl != nullptr);
+ m_identityNames = m_pibImpl->getIdentities();
}
IdentityContainer::const_iterator
@@ -116,8 +116,7 @@
{
if (m_identityNames.count(identityName) == 0) {
m_identityNames.insert(identityName);
- m_identities[identityName] =
- shared_ptr<detail::IdentityImpl>(new detail::IdentityImpl(identityName, m_pibImpl, true));
+ m_identities[identityName] = make_shared<detail::IdentityImpl>(identityName, m_pibImpl, true);
}
return get(identityName);
}
@@ -140,7 +139,7 @@
id = it->second;
}
else {
- id = shared_ptr<detail::IdentityImpl>(new detail::IdentityImpl(identityName, m_pibImpl, false));
+ id = make_shared<detail::IdentityImpl>(identityName, m_pibImpl, false);
m_identities[identityName] = id;
}
return Identity(id);