Code Review / govpp.git / commit
? search:
summary | shortlog | log | commit | commitdiff | review | tree
(parent: f4f4a86) | patch
socket adapter: don't bother sending sockclnt_delete messages 60/27860/3
authorDave Barach <dave@barachs.net>
Fri, 10 Jul 2020 16:19:01 +0000 (12:19 -0400)
committerDave Barach <dave@barachs.net>
Tue, 14 Jul 2020 14:25:48 +0000 (10:25 -0400)
commit1548c7e12531e3d055567d761c580a1c7ff0ac40
treea6a6d01ae6aa6e10c1420fd47a80bd29f989d729tree | snapshot
parentf4f4a8613711d724cd4dc7224e8d7a1de4b8050ecommit | diff
socket adapter: don't bother sending sockclnt_delete messages

vpp cleans up socket clients when the socket closes. I verified that
sending a correctly-marshalled sockclnt_delete message results in
duplicate close complaints.

The marshalling code wasn't right - the (important) client_index field
is in a non-standard position - and there's no point in sending
a message we don't need to send.

Signed-off-by: Dave Barach <dave@barachs.net>
Change-Id: Ia32980b08a99cc878835f5db21f64de391759fa4
adapter/socketclient/socketclient.go diff | blob | history
The GoVPP Gerrit repository has been archived.