Ah, hard for me to review ;-) Do you plan to commit it, or do you want me to look at the current implementation and try to fix that?
EdB On Tue, Apr 9, 2013 at 10:49 AM, Alex Harui <aha...@adobe.com> wrote: > Locally. > > > > > Sent via the PANTECH Discover, an AT&T 4G LTE smartphone. > > Erik de Bruin <e...@ixsoftware.nl> wrote: > > > Locally? Or did I miss a commit? > > EdB > > > > On Tue, Apr 9, 2013 at 8:10 AM, Alex Harui <aha...@adobe.com> wrote: >> I think I have copied the Falconjs implementation to falconjx. >> >> >> >> >> Sent via the PANTECH Discover, an AT&T 4G LTE smartphone. >> >> Erik de Bruin <e...@ixsoftware.nl> wrote: >> >> >> None of my test cases (and none of my real world code, I think they're >> lazy) uses wildcard imports, so I wasn't aware of this. But since they >> are a feature of AS, we need to be able to handle them. The compiler >> seems to be the proper handler for this, so I'll have a look at how >> FalconJS does this and see how we might re-use your solution. >> >> Added to the list. >> >> EdB >> >> >> >> On Mon, Apr 8, 2013 at 3:31 PM, Alex Harui <aha...@adobe.com> wrote: >>> Erik, >>> >>> I haven’t found the code yet, but the FlexJS output seems to be using >>> import statements as a way of setting up the goog.requires. The closure >>> compiler subsequently doesn’t like wild card imports >>> >>> import foo.bar.*; >>> >>> And I think it will miss same-package class dependencies, unless that is >>> being picked up somewhere else (and of course, will report unnecessary >>> requires if you haven’t cleaned up your imports. >>> >>> FalconJS built up a list of dependencies as they were discovered by the >>> compiler. This seems to be sufficient for the two test cases I’m working >>> with. Any objections to switching away from imports to something more like >>> how FalconJS does it? >>> >>> -- >>> Alex Harui >>> Flex SDK Team >>> Adobe Systems, Inc. >>> http://blogs.adobe.com/aharui >> >> >> >> -- >> Ix Multimedia Software >> >> Jan Luykenstraat 27 >> 3521 VB Utrecht >> >> T. 06-51952295 >> I. www.ixsoftware.nl > > > > -- > Ix Multimedia Software > > Jan Luykenstraat 27 > 3521 VB Utrecht > > T. 06-51952295 > I. www.ixsoftware.nl -- Ix Multimedia Software Jan Luykenstraat 27 3521 VB Utrecht T. 06-51952295 I. www.ixsoftware.nl