Basically not PAYG. FYI there are some ongoing tasks to reduce the PAYG debt. I don’t want to add to it. https://github.com/apache/royale-asjs/issues/641
Most of the code in Language.as gets stripped out if not used. If that’s not the case here, I’m going to ask some more questions. 1. Where/why is this code needed? You made a comment about XML not being available. Why would that be an issue? 2. Is there another way to achieve the same result that’s more PAYG? Thanks, Harbs > On Sep 17, 2021, at 4:44 AM, Edward Stangler <estang...@bradmark.com> wrote: > > > 204 bytes (~0.03% of small program) in JS release > > (presumably mostly due to the one new static function) > (develop vs. develop + just these Language changes) > (no difference between full rebuild or just Language rebuild) > > TestSetClipboard.swf > release - before: 87,634 > release - after: 87,634 > debug - before: 176,590 > debug - after: 176,590 > > TestSetClipboard JS directory > release - before: 620,221 > release - after: 620,425 > debug - before: 3,145,663 > debug - after: 3,146,567 > > > (Test case from issue #995) Test.swf > release - before: 121,852 > release - after: 121,777 > debug - before: 244,914 > debug - after: 244,916 > > (Test case from issue #995) Test JS directory > release - before: 831,174 > release - after: 831,378 > debug - before: 4,003,520 > debug - after: 4,004,424 > > > TestSetClipboard.mxml: > <?xml version="1.0" encoding="utf-8"?> > <mx:Application > xmlns:fx="http://ns.adobe.com/mxml/2009";<http://ns.adobe.com/mxml/2009> > xmlns:js="library://ns.apache.org/royale/express" > xmlns:mx="library://ns.apache.org/royale/mx"> > > <fx:Script> > <![CDATA[ > import mx.system.System; > > public function testSetClipboard() : void > { > System.setClipboard(" This has three spaces > in front."); > } > ]]> > </fx:Script> > > <mx:Button label="Go" click="testSetClipboard();" /> > </mx:Application> > > > > On 9/16/2021 12:25 PM, Harbs wrote: > > One question I had: > > Do the changes to Language.as add to application size for applications which > don't need it? > > > > On Sep 16, 2021, at 9:20 AM, Piotr Zarzycki wrote: > > This fixes seems to be helpful. Do you think we should merge them ? > > czw., 16 wrz 2021 o 00:41 estanglerbm > <notificati...@github.com><mailto:notificati...@github.com> napisa?(a): > > > > Don't forget about these fixes. > > - > You are receiving this because you are subscribed to this thread. > Reply to this email directly, view it on GitHub > <https://github.com/apache/royale-asjs/pull/1127#issuecomment-920434990><https://github.com/apache/royale-asjs/pull/1127#issuecomment-920434990>, > or unsubscribe > <https://github.com/notifications/unsubscribe-auth/AF3K2K72XOPFFUX4P335WNLUCEOILANCNFSM5AFBH67Q><https://github.com/notifications/unsubscribe-auth/AF3K2K72XOPFFUX4P335WNLUCEOILANCNFSM5AFBH67Q> > . > > > > > > -- > > Piotr Zarzycki > > > > > >
