search

EngoEngine / ecs

A Go-implementation of the Entity-Component-System paradigm
MIT License
298 stars 43 forks source link

`...string` instead of `[]string` for NewEntity #11

Closed EtienneBruines closed 8 years ago

EtienneBruines commented 8 years ago

Harley Laue noted at the slack-chat, that our ecs.NewEntity function takes a []string as an argument.

We could easily change this to ...string, without changing much to the ecs.NewEntity method, it wouldn't change anything to the Entity type, and it would make the syntax look way nicer. 

e := ecs.NewEntity([]string{"RenderSystem", "MouseSystem"})

would become

e := ecs.NewEntity("RenderSystem", "MouseSystem")
EtienneBruines commented 8 years ago

I could change this, and everything that depends on it (wiki, demos, tutorials) once I have an OK.

paked commented 8 years ago

I've been thinking about this. Lets do it.

On Mon, Apr 11, 2016 at 9:24 AM Etienne Bruines notifications@github.com wrote:

I could change this, and everything that depends on it (wiki, demos, tutorials) once I have an OK.

— You are receiving this because you are subscribed to this thread. Reply to this email directly or view it on GitHub https://github.com/EngoEngine/ecs/issues/11#issuecomment-208090472

paked commented 8 years ago

Wooo! Progress.