From f670a59da8b93e6c3ec1eebf48297e3db639694c Mon Sep 17 00:00:00 2001
From: Davide Mor <davide.mor@skywarder.eu>
Date: Thu, 21 Jul 2022 23:10:17 +0200
Subject: [PATCH] [CI/CD] Fixed sx1278 lints
---
src/tests/drivers/sx1278/test-sx1278-bench.cpp | 17 +++++++++--------
src/tests/drivers/sx1278/test-sx1278-bidir.cpp | 5 +++--
2 files changed, 12 insertions(+), 10 deletions(-)
diff --git a/src/tests/drivers/sx1278/test-sx1278-bench.cpp b/src/tests/drivers/sx1278/test-sx1278-bench.cpp
index aa7ce891c..a907b802d 100644
--- a/src/tests/drivers/sx1278/test-sx1278-bench.cpp
+++ b/src/tests/drivers/sx1278/test-sx1278-bench.cpp
@@ -103,14 +103,15 @@ struct Stats
{
// Prints are REALLY slow, so take a COPY of stats, so we can print an
// instant in time.
- Stats stats = *this;
-
- printf("stats.last_recv_packet = %d\n", stats.last_recv_packet);
- printf("stats.corrupted_packets = %d\n", stats.corrupted_packets);
- printf("stats.send_count = %d\n", stats.send_count);
- printf("stats.recv_count = %d\n", stats.recv_count);
- printf("stats.recv_errors = %d\n", stats.recv_errors);
- printf("stats.packet_loss = %.2f %%\n", stats.packet_loss() * 100.0f);
+ Stats stats_now = *this;
+
+ printf("stats.last_recv_packet = %d\n", stats_now.last_recv_packet);
+ printf("stats.corrupted_packets = %d\n", stats_now.corrupted_packets);
+ printf("stats.send_count = %d\n", stats_now.send_count);
+ printf("stats.recv_count = %d\n", stats_now.recv_count);
+ printf("stats.recv_errors = %d\n", stats_now.recv_errors);
+ printf("stats.packet_loss = %.2f %%\n",
+ stats_now.packet_loss() * 100.0f);
}
} stats;
diff --git a/src/tests/drivers/sx1278/test-sx1278-bidir.cpp b/src/tests/drivers/sx1278/test-sx1278-bidir.cpp
index 1fcd851c6..174474ae6 100644
--- a/src/tests/drivers/sx1278/test-sx1278-bidir.cpp
+++ b/src/tests/drivers/sx1278/test-sx1278-bidir.cpp
@@ -108,7 +108,8 @@ void recvLoop()
char buf[64];
while (1)
{
- ssize_t res = sx1278->receive((uint8_t *)buf, sizeof(buf));
+ ssize_t res =
+ sx1278->receive(reinterpret_cast<uint8_t *>(buf), sizeof(buf));
if (res != -1)
{
// Make sure there is a terminator somewhere
@@ -127,7 +128,7 @@ void sendLoop(int interval, const char *data)
{
miosix::Thread::sleep(interval);
- sx1278->send((uint8_t *)buf, strlen(buf) + 1);
+ sx1278->send(reinterpret_cast<uint8_t *>(buf), strlen(buf) + 1);
printf("[sx1278] Sent '%s'\n", buf);
}
}
--
GitLab