face: use move semantics for sockets where possible
Change-Id: I2af595073f862c570c1ce0dcb3717f3d2b9cfd71
Refs: #2613
diff --git a/daemon/face/udp-factory.hpp b/daemon/face/udp-factory.hpp
index cfbd077..e59275d 100644
--- a/daemon/face/udp-factory.hpp
+++ b/daemon/face/udp-factory.hpp
@@ -30,10 +30,9 @@
#include "udp-channel.hpp"
#include "multicast-udp-face.hpp"
-
namespace nfd {
-// @todo The multicast support for ipv6 must be implemented
+/// @todo IPv6 multicast support not implemented
class UdpFactory : public ProtocolFactory
{
@@ -51,7 +50,7 @@
}
};
- typedef std::map< udp::Endpoint, shared_ptr<MulticastUdpFace> > MulticastFaceMap;
+ typedef std::map<udp::Endpoint, shared_ptr<MulticastUdpFace>> MulticastFaceMap;
explicit
UdpFactory(const std::string& defaultPort = "6363");
@@ -143,13 +142,13 @@
const std::string& multicastPort,
const std::string& networkInterfaceName = "");
- // from Factory
+ // from ProtocolFactory
virtual void
createFace(const FaceUri& uri,
const FaceCreatedCallback& onCreated,
const FaceConnectFailedCallback& onConnectFailed);
- virtual std::list<shared_ptr<const Channel> >
+ virtual std::list<shared_ptr<const Channel>>
getChannels() const;
/**
@@ -159,7 +158,6 @@
getMulticastFaces() const;
PUBLIC_WITH_TESTS_ELSE_PRIVATE:
-
void
prohibitEndpoint(const udp::Endpoint& endpoint);
@@ -183,7 +181,6 @@
shared_ptr<UdpChannel>
findChannel(const udp::Endpoint& localEndpoint);
-
/**
* \brief Look up multicast UdpFace using specified local endpoint
*
@@ -196,7 +193,7 @@
findMulticastFace(const udp::Endpoint& localEndpoint);
PUBLIC_WITH_TESTS_ELSE_PRIVATE:
- typedef std::map< udp::Endpoint, shared_ptr<UdpChannel> > ChannelMap;
+ typedef std::map<udp::Endpoint, shared_ptr<UdpChannel>> ChannelMap;
ChannelMap m_channels;
MulticastFaceMap m_multicastFaces;
@@ -205,14 +202,12 @@
std::set<udp::Endpoint> m_prohibitedEndpoints;
};
-
inline const UdpFactory::MulticastFaceMap&
UdpFactory::getMulticastFaces() const
{
return m_multicastFaces;
}
-
} // namespace nfd
#endif // NFD_DAEMON_FACE_UDP_FACTORY_HPP