btc-ag / service-idl

Xtext-based Service IDL (Interface Definition Language) and Code Generators for Protobuf, C++, Java and .NET
Eclipse Public License 2.0
8 stars 8 forks source link

Allow configuration of generated module names #164

Closed sigiesec closed 5 years ago

sigiesec commented 6 years ago

For example, instead of using "ServiceAPI", only "API" might be used.

sigiesec commented 6 years ago

When using "API" instead of "ServiceAPI", it is no longer clear from the namespace that the contained structs are non-opaque "data transfer objects". In that case, this information should be added to the names of the classes as a suffix, or a subnamepace "DTO" might be used. But maybe it would be better just to stick with "ServiceAPI".

kamiddel commented 5 years ago

Meanwhile I am of similar opinion. In respect of the usage, the name ServiceAPI makes sense. ... So, I think there is no need to have a way to configure the namespace to "API".

sigiesec commented 5 years ago

Thanks for your response. I am closing the issue consequently.