search

Mirai-Team / mirai-project

C++ library for games making purposes.
Other
3 stars 0 forks source link

Added EventManager #49

Closed Lomadriel closed 9 years ago

Lomadriel commented 9 years ago

I think now Event Manager is functional.

See #38

Lomadriel commented 9 years ago

I must add some documentation before we can merge.

Lomadriel commented 9 years ago

Done.

Lomadriel commented 9 years ago

We should also add deleteListener() and clearListener() methods.

Lomadriel commented 9 years ago

Done

CBenoit commented 9 years ago

I'm going to test the example and read all the code then confirm the merge.

Lomadriel commented 9 years ago

So I think we can confirm the merge ?

CBenoit commented 9 years ago

There is still this issue : https://github.com/Mirai-Team/mirai-project/issues/38#issuecomment-76596282 And by the way, it seems that you "only" did a communication system (broadcast), not really an event manager. Or at least, it's not what I think it would be. Let me some time to think about it.

Lomadriel commented 9 years ago

CBenoit:

And by the way, it seems that you "only" did a communication system (broadcast), not really an event manager. Or at least, it's not what I think it would be. Let me some time to think about it.

Yes, it's more a broadcasting system than an event manager.

For the warning, I don't know how to correct it properly.

Lomadriel commented 9 years ago

I have find a way but I need some time to finish it.

Lomadriel commented 9 years ago

Warning corrected.

Lomadriel commented 9 years ago

Exemple improved.