Hey @fwilleke80,
you are right, it does indicate an error, hence my example from above. When I searched in our internal code base for similar code, I found a lot of
return NodeData::GetDDescription(node, description, flags);
calls. While there are certainly node types which customize their description handling, this seems to be more a precaution for the case when we decide that we want to implement something in NodeData::GetDDescription in the future. Or something has been there in the past and we optimized it away. In any case, one should not take my comment about calling the base implementation being pointless too seriously. It is best to leave the call in, although it currently has no practical effect and probably will never have.
Cheers,
Ferdinand