Skip to content

add a missing (required) argument to udp_socket#227

Closed
forvev wants to merge 1 commit intofrankarobotics:mainfrom
forvev:main
Closed

add a missing (required) argument to udp_socket#227
forvev wants to merge 1 commit intofrankarobotics:mainfrom
forvev:main

Conversation

@forvev
Copy link
Copy Markdown

@forvev forvev commented Mar 10, 2026

Couldn't build the ros workspace as the second argument was missing. According to the Poco repo, DatatagramSocket indeed requires the second argument as shown below:
DatagramSocket(const SocketAddress& address, bool reuseAddress, bool reusePort = false, bool ipV6Only = false);

Couldn't build the ros workspace as the second argument was missing. According to the Poco repo, DatatagramSocket indeed requires the second argument as shown below: DatagramSocket(const SocketAddress& address, bool reuseAddress, bool reusePort = false, bool ipV6Only = false);
@francando francando self-requested a review April 24, 2026 14:32
@francando
Copy link
Copy Markdown
Contributor

Duplicate of #126

@francando francando marked this as a duplicate of #126 Apr 24, 2026
@francando francando closed this Apr 24, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants