Re: [apache/royale-asjs] Various small fixes and changes (#1127)

[Edward Stangler]

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