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
>

Reply via email to