wboayue / rust-ibapi / 21815270884

86%

Build Type

push

github

Committed by web-flow

Commit Message

Add ConnectionOptions with TCP_NODELAY support (#394)

* Add ConnectionOptions struct with TCP_NODELAY support

Replace per-call-site env var approach with a fluent builder API.
ConnectionOptions absorbs tcp_no_delay and startup_callback into a
single options struct. Existing connect/connect_with_callback methods
delegate to new connect_with_options on both sync and async clients.

TCP_NODELAY is applied on initial connect and reconnect for both
sync (TcpSocket) and async (AsyncConnection) paths.

* Add From<Option<StartupMessageCallback>> for ConnectionOptions

Dedup callback-to-options conversion in 3 call sites.

* Avoid unnecessary Arc clone, expand connect_with_options doc

* Move transport setup from sync Client to Connection layer

Push TcpStream creation and TCP_NODELAY into Connection<TcpSocket>::connect_with_options
and TcpSocket::connect, mirroring the async delegation pattern.

* Extract shared init() in sync Connection, unify callback type

- All connect variants converge on init() instead of chaining
- Revert connect_with_callback to Box (matching async signature)
- Remove Arc dependency from sync connection

* Store ConnectionOptions in AsyncConnection, extract connect_socket helper

Run Details

45 of 50 new or added lines in 6 files covered. (90.0%)

2 existing lines in 2 files now uncovered.

8811 of 10211 relevant lines covered (86.29%)

118.03 hits per line