Broken support for FW_OBJECT_NAMES == 0 about cookiecutter-fprime-deployment HOT 7 CLOSED

The fix that is currently on the devel branch should correct that, I'll wait on other updates to push it to master.

from cookiecutter-fprime-deployment.

Well, thats a bug. The proposed fix is also buggy, but I agree that it should follow the pattern in the .cpp file. The cheetah template from F Prime upstream looks like this:

  // ----------------------------------------------------------------------
  // Construction, initialization, and destruction
  // ----------------------------------------------------------------------

  ${name}ComponentImpl ::
\#if FW_OBJECT_NAMES == 1
    ${name}ComponentImpl(
$emit_non_port_params([ $param_compName ])
    ) :
      ${component_base}(compName)
\#else
    ${name}ComponentImpl(void)
\#endif
  {

  }

So I think I will use that pattern.

from cookiecutter-fprime-deployment.

I think the fix actually ends up being this:

  {{cookiecutter.component_slug}}{{cookiecutter.component_explicit_component_suffix}}{{cookiecutter.component_impl_suffix}} ::
#if FW_OBJECT_NAMES == 1
    {{cookiecutter.component_slug}}{{cookiecutter.component_explicit_component_suffix}}{{cookiecutter.component_impl_suffix}}(
        const char *const compName
    ) :
      {{cookiecutter.component_slug}}ComponentBase(compName),
#else
      {{cookiecutter.component_slug}}{{cookiecutter.component_explicit_component_suffix}}{{cookiecutter.component_impl_suffix}}(void),
#endif

from cookiecutter-fprime-deployment.

I think it should not have , at the end of the alternate constructor header definition. Or does that work?

from cookiecutter-fprime-deployment.

I believe you need the comma for constructor initialization of member variables, which I am using. So, yes I think it works just fine.

from cookiecutter-fprime-deployment.

I just checked and that gives:

error: expected unqualified-id before '{' token

, is not a valid character to come directly after the constructor header definition. You could put : but only if followed by a member variable constructor initialization. Otherwise, it gives the same error as above. , is only meant for separating member variables in the constructor initialization section.

from cookiecutter-fprime-deployment.

Oh, I see, you are correct, the #if segment has ) : but I messed up the #else segment by putting ),.

from cookiecutter-fprime-deployment.