Do not use boost::optional for TcpSocket

Older boost versions require T to be copy-constructable in optional<T>. This
is not the case for TcpSocket, which is only move-constructable.

Fixes compilation on debian jessie

1 files changed, 3 insertions, 3 deletions

diff --git a/src/dabOutput/dabOutputTcp.cpp b/src/dabOutput/dabOutputTcp.cpp
index 94d379e..975fdff 100644
--- a/src/dabOutput/dabOutputTcp.cpp
+++ b/src/dabOutput/dabOutputTcp.cpp
@@ -133,9 +133,9 @@ class TCPDataDispatcher
 
                 while (m_running) {
                     // Add a new TCPConnection to the list, constructing it from the client socket
-                    auto optional_sock = m_listener_socket.accept(timeout_ms);
-                    if (optional_sock) {
-                        m_connections.emplace(m_connections.begin(), std::move(*optional_sock));
+                    auto sock = m_listener_socket.accept(timeout_ms);
+                    if (sock.isValid()) {
+                        m_connections.emplace(m_connections.begin(), move(sock));
                     }
                 }
             }