search

CNES / ccsdsmo-malgo

CCSDS MO MAL Go API
https://cnes.github.io/ccsdsmo-malgo
MIT License
10 stars 1 forks source link

Extend ElementList interface #6

Closed etiennelndr closed 6 years ago

etiennelndr commented 6 years ago

I may have found an oversight. Indeed, all your lists (LongList, IntegerList,...) doesn't implement the method composite.Composite(). Is it normal ? Because, in my opinion, it is needed to extend the ElementList interface.

freyssin commented 6 years ago

You are true. All these structures should be generated automatically from CCSDS XML specifications (MAL, COM, Common)..

ochurlaud commented 6 years ago

@freyssin : just for your info (you might know it) if you had writtent "Fixes #6" or "Closes #6" in your commit, the issue would have been automatically closed with a link to the commit.

See https://help.github.com/articles/closing-issues-using-keywords/