Skip to content

Commit

Permalink
Revise String.hpp
Browse files Browse the repository at this point in the history
Remove explicit from char* constructor
The autocoder depends on the implicit constructor
  • Loading branch information
bocchino committed Apr 29, 2024
1 parent d784350 commit 172ebb0
Show file tree
Hide file tree
Showing 2 changed files with 6 additions and 10 deletions.
2 changes: 1 addition & 1 deletion Fw/Types/String.hpp
Original file line number Diff line number Diff line change
Expand Up @@ -28,7 +28,7 @@ class String final : public StringBase {

explicit String(const StringBase& src) : StringBase() { *this = src; }

Check warning

Code scanning / CodeQL

Unchecked return value Warning

The return value of non-void function
operator=
is not checked.

Check warning

Code scanning / CodeQL

Unchecked function argument Warning

This use of parameter src has not been checked.

Check notice

Code scanning / CodeQL

More than one statement per line Note

This line contains 3 statements; only one is allowed.

explicit String(const char* src) : StringBase() { *this = src; }
String(const char* src) : StringBase() { *this = src; }

~String() {}

Expand Down
14 changes: 5 additions & 9 deletions Ref/SignalGen/SignalGen.hpp
Original file line number Diff line number Diff line change
Expand Up @@ -40,20 +40,16 @@ namespace Ref {
F32 Amplitude,
F32 Phase,
Ref::SignalType SigType
);
) final;

void SignalGen_Toggle_cmdHandler(
FwOpcodeType opCode, /*!< The opcode*/
U32 cmdSeq /*!< The command sequence number*/
);
) final;
void SignalGen_Skip_cmdHandler(
FwOpcodeType opCode, /*!< The opcode*/
U32 cmdSeq /*!< The command sequence number*/
);
void SignalGen_GenerateArray_cmdHandler(
FwOpcodeType opCode, /*!< The opcode*/
U32 cmdSeq /*!< The command sequence number*/
);
) final;

//! Handler implementation for command SignalGen_Dp
//!
Expand All @@ -62,7 +58,7 @@ namespace Ref {
FwOpcodeType opCode, //!< The opcode
U32 cmdSeq, //!< The command sequence number
U32 records
) override;
) final;

// ----------------------------------------------------------------------
// Handler implementations for data products
Expand All @@ -72,7 +68,7 @@ namespace Ref {
void dpRecv_DataContainer_handler(
DpContainer& container, //!< The container
Fw::Success::T status //!< The container status
) override;
) final;


public:
Expand Down

0 comments on commit 172ebb0

Please sign in to comment.