But upon further consideration I agree that there is no sense in defining this as a constant.
In general I am against replacing numeric constants with magic numbers unless it is something super-obvious like 1 or just something used in tests. It's not just about conveniently refactoring code…
Using clippy is a great suggestion, but the only warnings it produces regarding files in src/link/* are about dead-code. "unreal_config" code might be deleted (moved out into another project), so I…
MessageReader implementation
В приложении слегка обновленный файл с SRS.
Я не вижу смысла использовать что-то кроме tcp/ip, поэтому про udp (и…
serial was removed as a dependency, tests were rewritten to use separate temporary files.
Unnecessary files are cleaned up with auxiliary struct with the drop trait, which seems to work just fine and does not require another dependency. Maybe just keep this solution or is tempfile superior in some way?
Do we really need to move test data into the code? I feel like separating them is a cleaner approach.
It sounds reasonable, but maybe wait until we have release version ready? Compile times do not seem to be an issue yet (and are unlikely to be in a foreseeable future), so it can save us some fuss.