From dc8062b20050b9dca14536ace687e95450a3e67b Mon Sep 17 00:00:00 2001 From: Andrew Stitcher Date: Thu, 21 Jan 2010 06:20:07 +0000 Subject: Throw sync connect failures straight out of AsynchConnector constructor git-svn-id: https://svn.apache.org/repos/asf/qpid/trunk/qpid@901555 13f79535-47bb-0310-9956-ffa450edef68 --- cpp/src/qpid/sys/posix/AsynchIO.cpp | 21 ++------------------- 1 file changed, 2 insertions(+), 19 deletions(-) (limited to 'cpp/src/qpid') diff --git a/cpp/src/qpid/sys/posix/AsynchIO.cpp b/cpp/src/qpid/sys/posix/AsynchIO.cpp index beb647afe6..d4c64dd814 100644 --- a/cpp/src/qpid/sys/posix/AsynchIO.cpp +++ b/cpp/src/qpid/sys/posix/AsynchIO.cpp @@ -144,12 +144,10 @@ class AsynchConnector : public qpid::sys::AsynchConnector, private: void connComplete(DispatchHandle& handle); - void syncFailure(); private: ConnectedCallback connCallback; FailedCallback failCallback; - std::string errMsg; const Socket& socket; public: @@ -176,21 +174,13 @@ AsynchConnector::AsynchConnector(const Socket& s, { socket.setNonblocking(); SocketAddress sa(hostname, boost::lexical_cast(port)); - try { - socket.connect(sa); - } catch(std::exception& e) { - // Defer reporting failure till we start polling - errMsg = e.what(); - } + // Note, not catching any exceptions here, also has effect of destructing + socket.connect(sa); } void AsynchConnector::start(Poller::shared_ptr poller) { startWatch(poller); - // If we previously detected an error insert failure callback now - if ( !errMsg.empty() ) { - DispatchHandle::call(boost::bind(&AsynchConnector::syncFailure, this)); - } } void AsynchConnector::connComplete(DispatchHandle& h) @@ -205,13 +195,6 @@ void AsynchConnector::connComplete(DispatchHandle& h) DispatchHandle::doDelete(); } -void AsynchConnector::syncFailure() -{ - DispatchHandle::stopWatch(); - failCallback(socket, -1, errMsg); - DispatchHandle::doDelete(); -} - /* * POSIX version of AsynchIO reader/writer * -- cgit v1.2.1