esa / CCSDS_MO_StubGenerator

Generates technology bindings for CCSDS MAL service specifications
Other
4 stars 9 forks source link

Document generator and COM object references #19

Open SamCooper opened 7 years ago

SamCooper commented 7 years ago

Would be nice if the tables for related and source objects said Parameter::ParameterIdentity rather than Parameter::1

CesarCoelho commented 7 years ago

Not for the related links because these can only link within the same service. This means that the information on the left hand side would be redundant (the service name): ThisServiceName::COMObject1 I believe that the number is enough for the related link because one can easily check the table and find it.

However, I think it is a good idea for the source link because it is able to link to "foreign" COM Objects (outside its service range) and therefore it could indeed be improved by following the conventions mentioned above: ServiceName::COMObjectName

CesarCoelho commented 7 years ago

In the XML files this value is represented as a numeric value and not as a string. Example:

This means that at the moment it would be impossible to "match" with the name of the respective COM object. The solution would be to exchange the number with a string name for the object, or support both the name and the number.

The source link table cell is generated here: https://github.com/esa/CCSDS_MO_StubGenerator/blob/master/GEN_DOCS/src/main/java/esa/mo/tools/stubgen/GeneratorDocx.java#L499

CesarCoelho commented 3 years ago

Moved to: https://github.com/esa/mo-services-java/issues/5

Please close this!