Updated parameter delegate to accept a serial buffer base instead of a parameter buffer

Brian Campuzano · Brian Campuzano · commit 538e93dcd80d · 2025-04-23T18:57:27.000-06:00
diff --git a/FppTest/component/active/test/ut/Tester.cpp b/FppTest/component/active/test/ut/Tester.cpp
@@ -131,7 +131,7 @@ Fw::SerializeStatus Tester::ActiveTestComponentBaseParamExternalDelegate ::
       const FwPrmIdType base_id,
       const FwPrmIdType local_id,
       const Fw::ParamValid prmStat,
-      Fw::ParamBuffer& buff
+      Fw::SerializeBufferBase& buff
   )
 {
   Fw::SerializeStatus stat;
@@ -176,7 +176,7 @@ Fw::SerializeStatus Tester::ActiveTestComponentBaseParamExternalDelegate ::
   serializeParam(
       const FwPrmIdType base_id,
       const FwPrmIdType local_id,
-      Fw::ParamBuffer& buff
+      Fw::SerializeBufferBase& buff
   ) const
 {
   Fw::SerializeStatus stat;
diff --git a/FppTest/component/active/test/ut/Tester.hpp b/FppTest/component/active/test/ut/Tester.hpp
@@ -372,14 +372,14 @@ class Tester : public ActiveTestGTestBase {
             const FwPrmIdType base_id, //!< The component base parameter ID to deserialize
             const FwPrmIdType local_id, //!< The parameter local ID to deserialize
             const Fw::ParamValid prmStat, //!< The parameter validity status
-            Fw::ParamBuffer& buff //!< The buffer containing the parameter to deserialize
+            Fw::SerializeBufferBase& buff //!< The buffer containing the parameter to deserialize
         ) override;
 
         //! Parameter serialization function for external parameter unit testing
         Fw::SerializeStatus serializeParam(
             const FwPrmIdType base_id, //!< The component base parameter ID to serialize
             const FwPrmIdType local_id, //!< The parameter local ID to serialize
-            Fw::ParamBuffer& buff //!< The buffer to serialize the parameter into
+            Fw::SerializeBufferBase& buff //!< The buffer to serialize the parameter into
         ) const override;
 
     };
diff --git a/FppTest/component/passive/test/ut/Tester.cpp b/FppTest/component/passive/test/ut/Tester.cpp
@@ -129,7 +129,7 @@ Fw::SerializeStatus Tester::PassiveTestComponentBaseParamExternalDelegate ::
       const FwPrmIdType base_id,
       const FwPrmIdType local_id,
       const Fw::ParamValid prmStat,
-      Fw::ParamBuffer& buff
+      Fw::SerializeBufferBase& buff
   )
 {
   Fw::SerializeStatus stat;
@@ -174,7 +174,7 @@ Fw::SerializeStatus Tester::PassiveTestComponentBaseParamExternalDelegate ::
   serializeParam(
       const FwPrmIdType base_id,
       const FwPrmIdType local_id,
-      Fw::ParamBuffer& buff
+      Fw::SerializeBufferBase& buff
   ) const
 {
   Fw::SerializeStatus stat;
@@ -213,4 +213,4 @@ Fw::SerializeStatus Tester::PassiveTestComponentBaseParamExternalDelegate ::
   }
 
   return stat;
-}
+}
diff --git a/FppTest/component/passive/test/ut/Tester.hpp b/FppTest/component/passive/test/ut/Tester.hpp
@@ -351,14 +351,14 @@ class Tester : public PassiveTestGTestBase {
             const FwPrmIdType base_id, //!< The component base parameter ID to deserialize
             const FwPrmIdType local_id, //!< The parameter local ID to deserialize
             const Fw::ParamValid prmStat, //!< The parameter validity status
-            Fw::ParamBuffer& buff //!< The buffer containing the parameter to deserialize
+            Fw::SerializeBufferBase& buff //!< The buffer containing the parameter to deserialize
         ) override;
 
         //! Parameter serialization function for external parameter unit testing
         Fw::SerializeStatus serializeParam(
             const FwPrmIdType base_id, //!< The component base parameter ID to serialize
             const FwPrmIdType local_id, //!< The parameter local ID to serialize
-            Fw::ParamBuffer& buff //!< The buffer to serialize the parameter into
+            Fw::SerializeBufferBase& buff //!< The buffer to serialize the parameter into
         ) const override;
 
     };
diff --git a/FppTest/component/queued/test/ut/Tester.cpp b/FppTest/component/queued/test/ut/Tester.cpp
@@ -129,7 +129,7 @@ Fw::SerializeStatus Tester::QueuedTestComponentBaseParamExternalDelegate ::
       const FwPrmIdType base_id,
       const FwPrmIdType local_id,
       const Fw::ParamValid prmStat,
-      Fw::ParamBuffer& buff
+      Fw::SerializeBufferBase& buff
   )
 {
   Fw::SerializeStatus stat;
@@ -174,7 +174,7 @@ Fw::SerializeStatus Tester::QueuedTestComponentBaseParamExternalDelegate ::
   serializeParam(
       const FwPrmIdType base_id,
       const FwPrmIdType local_id,
-      Fw::ParamBuffer& buff
+      Fw::SerializeBufferBase& buff
   ) const
 {
   Fw::SerializeStatus stat;
@@ -213,4 +213,4 @@ Fw::SerializeStatus Tester::QueuedTestComponentBaseParamExternalDelegate ::
   }
 
   return stat;
-}
+}
diff --git a/FppTest/component/queued/test/ut/Tester.hpp b/FppTest/component/queued/test/ut/Tester.hpp
@@ -373,14 +373,14 @@ class Tester : public QueuedTestGTestBase {
             const FwPrmIdType base_id, //!< The component base parameter ID to deserialize
             const FwPrmIdType local_id, //!< The parameter local ID to deserialize
             const Fw::ParamValid prmStat, //!< The parameter validity status
-            Fw::ParamBuffer& buff //!< The buffer containing the parameter to deserialize
+            Fw::SerializeBufferBase& buff //!< The buffer containing the parameter to deserialize
         ) override;
 
         //! Parameter serialization function for external parameter unit testing
         Fw::SerializeStatus serializeParam(
             const FwPrmIdType base_id, //!< The component base parameter ID to serialize
             const FwPrmIdType local_id, //!< The parameter local ID to serialize
-            Fw::ParamBuffer& buff //!< The buffer to serialize the parameter into
+            Fw::SerializeBufferBase& buff //!< The buffer to serialize the parameter into
         ) const override;
 
     };
diff --git a/Fw/Prm/PrmExternalTypes.hpp b/Fw/Prm/PrmExternalTypes.hpp
@@ -33,7 +33,7 @@ namespace Fw {
                 const FwPrmIdType base_id,
                 const FwPrmIdType local_id,
                 const ParamValid prmStat,
-                ParamBuffer& buff) = 0;
+                SerializeBufferBase& buff) = 0;
 
             //! Serialize a parameter into a parameter buffer
             //!
@@ -45,9 +45,9 @@ namespace Fw {
             virtual SerializeStatus serializeParam(
                 const FwPrmIdType base_id,
                 const FwPrmIdType local_id,
-                ParamBuffer& buff) const = 0;
+                SerializeBufferBase& buff) const = 0;
         };
 
 }  // namespace Fw
 
-#endif
+#endif
diff --git a/requirements.txt b/requirements.txt
@@ -20,19 +20,19 @@ fprime-fpl-convert-xml==1.0.3
 fprime-fpl-extract-xml==1.0.3
 fprime-fpl-layout==1.0.3
 fprime-fpl-write-pic==1.0.3
-fprime-fpp-check==3.0.0a8-67-gefde0d459
-fprime-fpp-depend==3.0.0a8-67-gefde0d459
-fprime-fpp-filenames==3.0.0a8-67-gefde0d459
-fprime-fpp-format==3.0.0a8-67-gefde0d459
-fprime-fpp-from-xml==3.0.0a8-67-gefde0d459
-fprime-fpp-locate-defs==3.0.0a8-67-gefde0d459
-fprime-fpp-locate-uses==3.0.0a8-67-gefde0d459
-fprime-fpp-syntax==3.0.0a8-67-gefde0d459
-fprime-fpp-to-cpp==3.0.0a8-67-gefde0d459
-fprime-fpp-to-dict==3.0.0a8-67-gefde0d459
-fprime-fpp-to-json==3.0.0a8-67-gefde0d459
-fprime-fpp-to-xml==3.0.0a8-67-gefde0d459
-fprime-fpp-to-layout==3.0.0a8-67-gefde0d459
+fprime-fpp-check==3.0.0a8-71-g77dee8489
+fprime-fpp-depend==3.0.0a8-71-g77dee8489
+fprime-fpp-filenames==3.0.0a8-71-g77dee8489
+fprime-fpp-format==3.0.0a8-71-g77dee8489
+fprime-fpp-from-xml==3.0.0a8-71-g77dee8489
+fprime-fpp-locate-defs==3.0.0a8-71-g77dee8489
+fprime-fpp-locate-uses==3.0.0a8-71-g77dee8489
+fprime-fpp-syntax==3.0.0a8-71-g77dee8489
+fprime-fpp-to-cpp==3.0.0a8-71-g77dee8489
+fprime-fpp-to-dict==3.0.0a8-71-g77dee8489
+fprime-fpp-to-json==3.0.0a8-71-g77dee8489
+fprime-fpp-to-xml==3.0.0a8-71-g77dee8489
+fprime-fpp-to-layout==3.0.0a8-71-g77dee8489
 fprime-gds==3.6.1
 fprime-tools==3.6.1
 fprime-visual==1.0.2

Original file line number	Diff line number	Diff line change
`@@ -131,7 +131,7 @@ Fw::SerializeStatus Tester::ActiveTestComponentBaseParamExternalDelegate ::`
`131`	`131`	`const FwPrmIdType base_id,`
`132`	`132`	`const FwPrmIdType local_id,`
`133`	`133`	`const Fw::ParamValid prmStat,`
`134`		`- Fw::ParamBuffer& buff`
	`134`	`+ Fw::SerializeBufferBase& buff`
`135`	`135`	`)`
`136`	`136`	`{`
`137`	`137`	`Fw::SerializeStatus stat;`
`@@ -176,7 +176,7 @@ Fw::SerializeStatus Tester::ActiveTestComponentBaseParamExternalDelegate ::`
`176`	`176`	`serializeParam(`
`177`	`177`	`const FwPrmIdType base_id,`
`178`	`178`	`const FwPrmIdType local_id,`
`179`		`- Fw::ParamBuffer& buff`
	`179`	`+ Fw::SerializeBufferBase& buff`
`180`	`180`	`) const`
`181`	`181`	`{`
`182`	`182`	`Fw::SerializeStatus stat;`
Original file line number	Diff line number	Diff line change
`@@ -129,7 +129,7 @@ Fw::SerializeStatus Tester::PassiveTestComponentBaseParamExternalDelegate ::`
`129`	`129`	`const FwPrmIdType base_id,`
`130`	`130`	`const FwPrmIdType local_id,`
`131`	`131`	`const Fw::ParamValid prmStat,`
`132`		`- Fw::ParamBuffer& buff`
	`132`	`+ Fw::SerializeBufferBase& buff`
`133`	`133`	`)`
`134`	`134`	`{`
`135`	`135`	`Fw::SerializeStatus stat;`
`@@ -174,7 +174,7 @@ Fw::SerializeStatus Tester::PassiveTestComponentBaseParamExternalDelegate ::`
`174`	`174`	`serializeParam(`
`175`	`175`	`const FwPrmIdType base_id,`
`176`	`176`	`const FwPrmIdType local_id,`
`177`		`- Fw::ParamBuffer& buff`
	`177`	`+ Fw::SerializeBufferBase& buff`
`178`	`178`	`) const`
`179`	`179`	`{`
`180`	`180`	`Fw::SerializeStatus stat;`
`@@ -213,4 +213,4 @@ Fw::SerializeStatus Tester::PassiveTestComponentBaseParamExternalDelegate ::`
`213`	`213`	`}`
`214`	`214`
`215`	`215`	`return stat;`
`216`		`-}`
	`216`	`+}`