From bc4ee95004bc868c703bb0c44e40b6a8bb43c72b Mon Sep 17 00:00:00 2001 From: Peter Barker Date: Thu, 8 Jul 2021 16:10:27 +1000 Subject: [PATCH] ArduSub: remove misleading NOINLINE comment NOINLINE hasn't been used in years in these files --- ArduSub/GCS_Mavlink.cpp | 10 ---------- 1 file changed, 10 deletions(-) diff --git a/ArduSub/GCS_Mavlink.cpp b/ArduSub/GCS_Mavlink.cpp index 9c756a54cf..4de5f9894d 100644 --- a/ArduSub/GCS_Mavlink.cpp +++ b/ArduSub/GCS_Mavlink.cpp @@ -2,16 +2,6 @@ #include "GCS_Mavlink.h" -/* - * !!NOTE!! - * - * the use of NOINLINE separate functions for each message type avoids - * a compiler bug in gcc that would cause it to use far more stack - * space than is needed. Without the NOINLINE we use the sum of the - * stack needed for each message type. Please be careful to follow the - * pattern below when adding any new messages - */ - MAV_TYPE GCS_Sub::frame_type() const { return MAV_TYPE_SUBMARINE;