mavlink / MAVSDK / 10090112588

39%

Build Type

push

github

Committed by web-flow

Commit Message

core: Correctly close sockets (#2357)

* core: Correctly close sockets

* Update socket_holder.cpp - Fix newline style

* Update socket_holder.cpp

* Update src/mavsdk/core/tcp_client_connection.cpp

Co-authored-by: Jonas Vautherin <accounts@jonas.vautherin.ch>

* Update socket_holder.cpp - fix code style

* Update tcp_client_connection.cpp - fix code style

* Update tcp_server_connection.cpp - fix code style

* Update socket_holder.h - fix code style

* Update src/mavsdk/core/socket_holder.h

Co-authored-by: Julian Oes <julian@oes.ch>

* Update socket_holder.h - use 64 bit descriptor type on Win64

* Update socket_holder.cpp - use 64 bit descriptor type on Win64

* Update socket_holder.cpp - minor improving of if logic

* Remove default move constructor

It is currently not in use, and the default implementation is not suitable because it does not change _fd to INVALID for the object being copied from

---------

Co-authored-by: Jonas Vautherin <accounts@jonas.vautherin.ch>
Co-authored-by: Julian Oes <julian@oes.ch>

Run Details

45 of 46 new or added lines in 5 files covered. (97.83%)

47 existing lines in 5 files now uncovered.

11423 of 30485 relevant lines covered (37.47%)

255.27 hits per line