sbeTemplateId()?

[structurechart]

Hi,

Is there a reason that "static sbe_uint16_t sbeTemplateId()" is preferred to "static const sbe_uint16_t sbeTemplateId"?

With sbeTemplateId() being a regular static c98 function, it forbids users to use it as the value of the switch case.

"static constexpr sbe_uint16_t sbeTemplateId()" can be a good option but it seems like the project is sticking with c98.

Thanks in advance.

[tmontgomery]

There really isn't a reason. The generated codecs could do that. I'll mark it has an enhancement.

Just FYI, c98 is one of the codecs. But, a real c++11 codec would be nice to have.

[structurechart]

Hi,

I just tried and the change doesn't really help.

I tried to do something like:
switch (templateID) {
case Hearbeat::sbeTemplateId():
break;

But my GCC 4.6.x with -std=c++0x isn't happy about it either with an error message like this "sbeTemplateId() is not a constexpr function".

I guess I will have to create my own c11 codec.

Thanks for your help.

And sorry for the false suggestion.

[structurechart]

Just a FYI, static constexpr does work for my own purpose. I just tried.
Thanks.