Hey Mike, In my work on source maps, I've overridden a number of methods from ASEmitter that aren't currently in sub-emitters for JS. Should I go ahead and make new sub-emitters for those things?
- Josh On Tue, Apr 5, 2016 at 3:19 PM, Michael Schmalle <teotigraphix...@gmail.com> wrote: > On Tue, Apr 5, 2016 at 5:36 PM, bill <b...@beecavegames.com> wrote: > > > Here's our fork of the flex-falcon repo: > > > > https://github.com/Bee-Cave-Games/flex-falcon > > > > The last handful of commits reflect the changes Nimai mentioned above. > > We're > > still learning our way around the code, so some of these things could > > likely > > be done in a simpler / better way. > > > > > > > Looks good to me! > > Everything can be simpler, last year I went through the main javascript > emitter and broke every thing out into sub emitters, MethodEmitter etc. > > I did that because it was starting to turn into a rats nest, I think there > is to much inheritance right now but that is more design. So I created > composites so you could easily do what you are doing right now, work in the > Class/Method emitters and feel pretty safe you are only changing that code. > > I can say with a pretty good certainty that is the way most of those > emitters behave right now. There are a few places where there is still un > needed coupling and I think I still have those as TODOs. > > Mike > > > > > > > > > > > -- > > View this message in context: > > > http://apache-flex-development.2333347.n4.nabble.com/Starling-framework-compiled-to-JavaScript-tp52188p52192.html > > Sent from the Apache Flex Development mailing list archive at Nabble.com. > > >