Merge pull request #409 from rosflight/remove-nanoprintf

Replaced nanoprintf with vsnprintf
rosflight · Nov 15, 2023 · 8f8ea1c · 8f8ea1c
2 parents 1d20caa + ca26e3a
commit 8f8ea1c
Show file tree

Hide file tree

Showing 6 changed files with 4 additions and 393 deletions.
diff --git a/include/comm_manager.h b/include/comm_manager.h
@@ -35,7 +35,8 @@
 #include "interface/comm_link.h"
 #include "interface/param_listener.h"
 
-#include "nanoprintf.h"
+#include <stdarg.h>
+#include <stdio.h>
 
 #include <cstdint>
 #include <functional>

diff --git a/include/nanoprintf.h b/include/nanoprintf.h
diff --git a/scripts/rosflight.mk b/scripts/rosflight.mk
@@ -38,8 +38,7 @@ ROSFLIGHT_SRC = rosflight.cpp \
                 comm_manager.cpp \
                 command_manager.cpp \
                 rc.cpp \
-                mixer.cpp \
-                nanoprintf.cpp
+                mixer.cpp
 
 # Math Source Files
 VPATH := $(VPATH):$(TURBOMATH_DIR)

diff --git a/src/comm_manager.cpp b/src/comm_manager.cpp
@@ -408,7 +408,7 @@ void CommManager::log(CommLinkInterface::LogSeverity severity, const char* fmt,
   va_list args;
   va_start(args, fmt);
   char text[LOG_MSG_SIZE];
-  rosflight_firmware::nanoprintf::tfp_sprintf(text, fmt, args);
+  vsnprintf(text, LOG_MSG_SIZE, fmt, args);
   va_end(args);
 
   if (initialized_ && connected_)