Suggestion

[johnvh]

Good job, but I noticed a glaring issue:

You're coding against the concrete implementation of EventDispatcher.
This means that everything MUST extend EventDispatcher to work with
EventController. Fortunately, nearly everything (especially the native
flash lib) does. It is possible thought to not extend EventDispatcher
and instead implement the IEventDispatcher interface. Coding against
the interface will allow you to work with both cases, and should not
introduce any regressions (EventDispatcher just implements the
interface).

Thanks

[corbanb]

Man that's a GREAT idea! I think that should be a very easy change to
make. Let me take a quick look at it and I will see what we can do.
It's the early stages of the code so any and all great recommendations
like this are perfect! Thanks as always John!

[Ben Fhala]

love that, thanks :)
updated the svn to use the IEventDispatcher instead of the EventDispatcher.

Thanks great feedback will update the zip later but its up on the svn for now. anything that opens the door to more developers without drastic changes is sweet , thanks for looking so deeply into it and so quickly.

https://code.google.com/p/eventcontroller/source/checkout


--
Best regards,

Ben Fhala
http://fla.as/ec

--
--
http://fla.as/ec