Parameter deserialization delegation

[timcanham]

There have been discussions about delegating the deserialization of select parameters to outside libraries in the event they wish to keep their own copies and do their own deserialization.

Code changes:

Here is some sample generated code from Ref/RecvBuffApp/RecvBuffComponentAc.cpp, loadParameters() function:

    ...
    FwPrmIdType _id;

    _id = this->getIdBase() + PARAMID_PARAMETER1;

    // Get parameter parameter1
    this->m_param_parameter1_valid =
      this->m_ParamGet_OutputPort[0].invoke(
        _id,
        buff
      );

    // Deserialize value
    this->m_paramLock.lock();

    // If there was a deserialization issue, mark it invalid
    if (this->m_param_parameter1_valid == Fw::ParamValid::VALID) {
      stat = buff.deserialize(this->m_parameter1);
      if (stat != Fw::FW_SERIALIZE_OK) {
        this->m_param_parameter1_valid = Fw::ParamValid::DEFAULT;
        // Set default value
        this->m_parameter1 = 10;
      }
    }
    else {
      // Set default value
      this->m_param_parameter1_valid = Fw::ParamValid::DEFAULT;
      this->m_parameter1 = 10;
    }

    this->m_paramLock.unLock();

If the FPP could have a tag to specify that a certain parameter is delegated, the above code could be replaced with:

    ...
    FwPrmIdType _id;

    _id = this->getIdBase() + PARAMID_PARAMETER1;

    // Get parameter parameter1
    this->m_param_parameter1_valid =
      this->m_ParamGet_OutputPort[0].invoke(
        _id,
        buff
      );

    this->deserailizeParam(_id,buff);

where:

virtual void deserializeParam(const FwPrmIdType id, Fw::ParamBuffer& buff) = 0;

Would be overridden by the implementation component. The instance variable this->m_parameter1 could be removed from the component.

For saving a particular parameter, the code is currently:

  Fw::CmdResponse RecvBuffComponentBase ::
    paramSet_parameter1(Fw::SerializeBufferBase& val)
  {
    U32 _local_val;
    Fw::SerializeStatus _stat = val.deserialize(_local_val);
    if (_stat != Fw::FW_SERIALIZE_OK) {
      return Fw::CmdResponse::VALIDATION_ERROR;
    }

    // Assign value only if successfully deserialized
    this->m_paramLock.lock();
    this->m_parameter1 = _local_val;
    this->m_param_parameter1_valid = Fw::ParamValid::VALID;
    this->m_paramLock.unLock();

    // Call notifier
    this->parameterUpdated(PARAMID_PARAMETER1);
    return Fw::CmdResponse::OK;
  }

  Fw::CmdResponse RecvBuffComponentBase ::
    paramSet_parameter2(Fw::SerializeBufferBase& val)
  {
    I16 _local_val;
    Fw::SerializeStatus _stat = val.deserialize(_local_val);
    if (_stat != Fw::FW_SERIALIZE_OK) {
      return Fw::CmdResponse::VALIDATION_ERROR;
    }

    // Assign value only if successfully deserialized
    this->m_paramLock.lock();
    this->m_parameter2 = _local_val;
    this->m_param_parameter2_valid = Fw::ParamValid::VALID;
    this->m_paramLock.unLock();

    // Call notifier
    this->parameterUpdated(PARAMID_PARAMETER2);
    return Fw::CmdResponse::OK;
  }

The new function could be:

  Fw::CmdResponse RecvBuffComponentBase ::
    paramSave_parameter1()
  {
    if (this->m_ParamSet_OutputPort[0].isConnected()) {
      Fw::ParamBuffer saveBuff;

      FwPrmIdType id = 0;
      id = this->getIdBase() + PARAMID_PARAMETER1;

      this->serializeParam(_id,saveBuff);

      // Save the parameter
      this->m_ParamSet_OutputPort[0].invoke(
        id,
        saveBuff
      );

      return Fw::CmdResponse::OK;
    }

    return Fw::CmdResponse::EXECUTION_ERROR;
  }

where:

virtual void serializeParam(const FwPrmIdType id, Fw::ParamBuffer& buff) = 0;