From 868f9cebb08c3e1e37275078256c75bd63e456bf Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Beat=20K=C3=BCng?= Date: Fri, 13 Aug 2021 14:08:33 +0200 Subject: [PATCH] protocol_splitter: return 0 when no data to read instead of -1 This avoids mavlink adding another usleep() due to read error --- src/drivers/protocol_splitter/protocol_splitter.cpp | 8 ++++---- 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/src/drivers/protocol_splitter/protocol_splitter.cpp b/src/drivers/protocol_splitter/protocol_splitter.cpp index 0bbe0d8721..53ca8a730a 100644 --- a/src/drivers/protocol_splitter/protocol_splitter.cpp +++ b/src/drivers/protocol_splitter/protocol_splitter.cpp @@ -366,7 +366,7 @@ pollevent_t DevCommon::poll_state(struct file *filp) int DevCommon::try_to_copy_data(char *buffer, size_t buflen, MessageType message_type) { if (buflen == 0) { - return -1; + return 0; } switch (message_type) { @@ -398,7 +398,7 @@ int DevCommon::try_to_copy_data(char *buffer, size_t buflen, MessageType message return len_to_copy; } else { - return -1; + return 0; } case MessageType::Rtps: @@ -429,14 +429,14 @@ int DevCommon::try_to_copy_data(char *buffer, size_t buflen, MessageType message return len_to_copy; } else { - return -1; + return 0; } break; default: - return -1; + return 0; } }