I am working through these now with the maven build, I found a few areas that needed attention - sorry. Yes I had expected this to work the same across the two builds as well.
On Wed, Sep 28, 2016 at 9:42 AM, Christofer Dutz <christofer.d...@c-ware.de> wrote: > Ok ... so I tracked down the problem to > > JSGoogDocEmitter > > there in line 344 in emitPublic, you seem to have changed the output. > > Should I adjust the testcase? I don't quite understand why the test > doesn't break in the Ant build, because the output should be wrong there > too ... > > > Chris > > > ________________________________ > Von: Greg Dove <greg.d...@gmail.com> > Gesendet: Dienstag, 27. September 2016 19:53:45 > An: dev@flex.apache.org > Betreff: Re: [FlexJS] Problem with some test cases (@export was changed to > @expose)? > > This was me, and was needed for reflection support into static members. > > @expose is supposed to be deprecated, but currently seems to be the only > option that works for statics. Josh discovered this a while back I think, > with static accessors. > > I will check this test now, I thought I had them all updated. > > cheers > Greg > > > > On Wed, Sep 28, 2016 at 3:46 AM, Christofer Dutz < > christofer.d...@c-ware.de> > wrote: > > > Hi, > > > > > > it seems some recent changes broke things in the test-suite. I am getting > > failures in TestFlexJSClass. > > > > For: > > > > > > org.apache.flex.compiler.internal.codegen.js.flexjs. > > TestFlexJSClass#testConstants > > > > > > and > > > > > > org.apache.flex.compiler.internal.codegen.js.flexjs. > > TestFlexJSClass#testMethods > > > > > > it seems @export was changed to @expose ... would be cool if someone who > > knows what happened here could fix that. > > > > > > Chris > > >
