Re: git commit: [flex-asjs] [refs/heads/develop] - CORS security. Allow auth credentials to be passed when using cross site calls. This is required as well as setting the Access-Control-Allow-Origin h

2017-04-08 Thread Alex Harui
On 4/8/17, 9:59 PM, "Greg Dove" wrote: >Just a general question about this type of thing: > >Because this is JS-only, does GCC eliminate it as deadcode in a release >version if the method is never used? I would expect that, but I have not >checked that yet... if it does then perhaps the PAYG co

Re: [FlexJS] internal classes based on other classes

2017-04-08 Thread Harbs
> On Apr 9, 2017, at 12:46 AM, Alex Harui wrote: > > I'll look into it. I thought I'd fixed that already. I assume > BackgroundManager isn't one of the goog.requires in FactoryComposer.js? I don’t remember, and I already modified my state of TLF. If you need me to check, I can try and revert

Re: [FlexJS] non-primitive static consts

2017-04-08 Thread Harbs
> On Apr 9, 2017, at 12:50 AM, Alex Harui wrote: > > I assume this is only a problem for folks using -remove-circulars? > I have not tested with remove-circulars truned off, so very possibly, yes. > I assume what is broken is the order of loaded files instead of something > else? Correct. Wit

Re: git commit: [flex-asjs] [refs/heads/develop] - CORS security. Allow auth credentials to be passed when using cross site calls. This is required as well as setting the Access-Control-Allow-Origin h

2017-04-08 Thread Greg Dove
Just a general question about this type of thing: Because this is JS-only, does GCC eliminate it as deadcode in a release version if the method is never used? I would expect that, but I have not checked that yet... if it does then perhaps the PAYG concern might be moot (love that word!) On Sun, A

Re: [FlexJS] non-primitive static consts

2017-04-08 Thread Alex Harui
I assume this is only a problem for folks using -remove-circulars? I assume what is broken is the order of loaded files instead of something else? Thanks, -Alex On 4/8/17, 9:46 PM, "Harbs" wrote: >Related issue: > >The recent changes to Falcon has broken UIDUtil which has the following: >priva

Re: git commit: [flex-asjs] [refs/heads/develop] - CORS security. Allow auth credentials to be passed when using cross site calls. This is required as well as setting the Access-Control-Allow-Origin h

2017-04-08 Thread Alex Harui
IMO, with PAYG, this would go in an extension of HTTPService. Not all apps will need CORS. Thanks, -Alex On 4/8/17, 8:26 PM, "jmcl...@apache.org" wrote: >Repository: flex-asjs >Updated Branches: > refs/heads/develop 11ef21aae -> 326d69791 > > >CORS security. Allow auth credentials to be passe

Re: [FlexJS] non-primitive static consts

2017-04-08 Thread Harbs
Related issue: The recent changes to Falcon has broken UIDUtil which has the following: private static const UIDBuffer:BinaryData = new BinaryData(); Unless something changes, this needs to be changed to a getter. > On Apr 8, 2017, at 11:40 PM, Harbs wrote: > > Currently initializing non-primi

Re: [FlexJS] internal classes based on other classes

2017-04-08 Thread Alex Harui
I'll look into it. I thought I'd fixed that already. I assume BackgroundManager isn't one of the goog.requires in FactoryComposer.js? And I assume you cleaned out old JS files? -Alex On 4/8/17, 9:15 PM, "Harbs" wrote: >The current state of Falcon seems to be okay so far, but I just bumped >in

Re: [FlexJS] non-primitive static consts

2017-04-08 Thread Alex Harui
Please provide some examples. On 4/8/17, 8:40 PM, "Harbs" wrote: >Currently initializing non-primitive static types are a big no-no in >FlexJS. It causes all kinds of javascript runtime errors due to >non-defined types depending on the order of loaded files. > >I think this is an area which need

[FlexJS] internal classes based on other classes

2017-04-08 Thread Harbs
The current state of Falcon seems to be okay so far, but I just bumped into an interesting problem. In TLF, there’s a class: FactoryComposer. FactoryComposer has an internal class called FactoryBackgroundManager which subclasses org.apache.flex.textLayout.elements.BackgroundManager. Because Fa

[FlexJS] non-primitive static consts

2017-04-08 Thread Harbs
Currently initializing non-primitive static types are a big no-no in FlexJS. It causes all kinds of javascript runtime errors due to non-defined types depending on the order of loaded files. I think this is an area which needs some improvement. The improvement can take two variations: 1. We ca