robotology-playground / robotology-template-library

0 stars 0 forks source link

[Discussion] Namespace #4

Open drdanz opened 5 years ago

drdanz commented 5 years ago

We should enclose all the library in a namespace. I also suggest we use c++11 inline namespace for versioning.

Proposals:

aerydna commented 5 years ago

currently the namespace used for the only class(stub) present is robotology.

drdanz commented 5 years ago

Personal opinion, it is a bit too long to write... It's fine with me anyway

claudiofantacci commented 5 years ago

Please: rtl. It's just what one would think after reading robotology-template-library, just as for the standard one.

aerydna commented 5 years ago

my only concern is that with rtl there can be collision with other namespace more easily. otherwise it will be fine also for me

claudiofantacci commented 5 years ago

Are there any other rtd library out there that are likely to clash with this one? Otherwise I would always prefer brevity over lengthness for a namespace.

aerydna commented 5 years ago

with all the possible interpretation of the acronym rtl like RunTimeLibrary, Real-TimeLibrary and so on, the risk of collision in the present or in the future is too high. I would exclude rtl. let's think some other abbreviation:

other?

claudiofantacci commented 5 years ago

with all the possible interpretation of the acronym rtl like RunTimeLibrary, Real-TimeLibrary and so on, the risk of collision in the present or in the future is too high. I would exclude rtl. let's think some other abbreviation:

I still think that we should just go for rtl, but I see your point.

I would like to propose robotl that is a sound combination of robot, template library and library. Otherwise rbtl.

drdanz commented 5 years ago

Another concern is that any robotology related namespace could collide with other robotology repository. Perhaps we should choose a "standard" robotology namespace and use a sub-namespace. Or perhaps it's just overkill

aerydna commented 5 years ago

i'm ok with both robotl and rbtl. @drdanz tiebreak!