wboayue / rust-ibapi / 22127633895

86%

push

github

Fix sync skip handling, remove dead retry infrastructure (#423)

* Fix sync skip handling, remove dead retry infrastructure

Rewrite sync next() as a loop handling Skip directly -- the previous
recursive approach never triggered (error was not stored for Skip)
and would have caused unbounded stack growth if it did.

Remove dead Retry variant and all retry infrastructure
(MAX_DECODE_RETRIES, RetryDecision, check_retry, should_retry_error)
since process_decode_result no longer produces Retry.

Clean up matching dead code in async next().

* Handle skip in try_next/next_timeout, scope test helpers

- Extract handle_response() -> NextAction to share decode/skip/error
  logic across next(), try_next(), and next_timeout()
- try_next() now loops on Skip instead of returning None
- next_timeout() loops on Skip with deadline tracking
- Move test struct/static inside test function body

31 of 50 new or added lines in 2 files covered. (62.0%)

8935 of 10445 relevant lines covered (85.54%)

121.12 hits per line