fix: fix infinite busy-loop in RPLidar _read_response on device disconnect

[andrewshab3]

Type of change

• Bug fix
• New feature
• Documentation improvement
• Bounty issue submission
• Other:

Changes

• _read_response hung forever when USB-LIDAR was disconnected — no timeout, no error handling on inWaiting()
• The worker multiprocessing.Process could never check control_queue for "STOP", making graceful shutdown impossible
• Added a 5s timeout via time.monotonic() deadline and OSError catch on inWaiting(), both raising RPLidarException which is already handled by rplidar_processor

Checklist

• Code complies with style guidelines
• Self-review completed
• Documentation updated (if applicable)
• Local testing completed
• Tests added/updated (if applicable)

[0xbyt4]

Hi @andrewshab3 , Thanks for contribution ! Could you add unit tests for the new behavior

[andrewshab3]

Hi @andrewshab3 , Thanks for contribution ! Could you add unit tests for the new behavior

Done, thanks!

[0xbyt4]

Hi @andrewshab3 , Thanks for contribution ! Could you add unit tests for the new behavior

Done, thanks!

please fix error

[codecov]

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ All tests successful. No failed tests found.

📢 Thoughts on this report? Let us know!