You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@royale.apache.org by ca...@apache.org on 2018/10/20 19:13:22 UTC

[royale-asjs] branch develop updated: fix few GCC warnings

This is an automated email from the ASF dual-hosted git repository.

carlosrovira pushed a commit to branch develop
in repository https://gitbox.apache.org/repos/asf/royale-asjs.git


The following commit(s) were added to refs/heads/develop by this push:
     new 18912bd  fix few GCC warnings
18912bd is described below

commit 18912bd0a9a9eff68cc286494aa49f7ae01869be
Author: Carlos Rovira <ca...@apache.org>
AuthorDate: Sat Oct 20 21:13:15 2018 +0200

    fix few GCC warnings
---
 .../mx/rpc/remoting/CompressedRemoteObject.as      | 22 +++++++++++-----------
 .../src/main/royale/mx/rpc/remoting/Operation.as   | 22 +++++++++++-----------
 .../src/main/royale/mx/utils/ObjectUtil.as         |  2 +-
 3 files changed, 23 insertions(+), 23 deletions(-)

diff --git a/frameworks/projects/MXRoyale/src/main/royale/mx/rpc/remoting/CompressedRemoteObject.as b/frameworks/projects/MXRoyale/src/main/royale/mx/rpc/remoting/CompressedRemoteObject.as
index 951b1e8..2097769 100644
--- a/frameworks/projects/MXRoyale/src/main/royale/mx/rpc/remoting/CompressedRemoteObject.as
+++ b/frameworks/projects/MXRoyale/src/main/royale/mx/rpc/remoting/CompressedRemoteObject.as
@@ -122,21 +122,21 @@ package mx.rpc.remoting
                 if (included && excludeClasses && excludeClasses.length > 0) {
                     included = excludeClasses.indexOf(parameterClassName) == -1;
                 }
-                if (included) {
-                    COMPILE::SWF{
-                    var byteArray:ByteArray = new ByteArray();
-                    byteArray.writeObject(parameters[i]);
-                    byteArray.compress();
-                    parameters[i] = byteArray;
-                    }
-                    COMPILE::JS
-                    {
+                // if (included) {
+                //     COMPILE::SWF{
+                //     var byteArray:ByteArray = new ByteArray();
+                //     byteArray.writeObject(parameters[i]);
+                //     byteArray.compress();
+                //     parameters[i] = byteArray;
+                //     }
+                //     COMPILE::JS
+                //     {
                     // var bytearray:AMFBinaryData = new AMFBinaryData();
                     // bytearray.writeObject(parameter[i]);
                     // window["pako"].deflate(bytearray)
                     // parameter[i] = bytearray;
-                    }
-                }
+                //     }
+                // }
             }
             return parameters;
         }
diff --git a/frameworks/projects/MXRoyale/src/main/royale/mx/rpc/remoting/Operation.as b/frameworks/projects/MXRoyale/src/main/royale/mx/rpc/remoting/Operation.as
index f9d6b5c..85b2507 100644
--- a/frameworks/projects/MXRoyale/src/main/royale/mx/rpc/remoting/Operation.as
+++ b/frameworks/projects/MXRoyale/src/main/royale/mx/rpc/remoting/Operation.as
@@ -203,7 +203,7 @@ public class Operation extends AbstractOperation
     //---------------------------------
 
     /**
-     * @inheritDoc
+     * inheritDoc
      *  
      *  @langversion 3.0
      *  @playerversion Flash 9
@@ -265,7 +265,7 @@ public class Operation extends AbstractOperation
     }
 
     /**
-     * @inheritDoc
+     * inheritDoc
      *  
      *  @langversion 3.0
      *  @playerversion Flash 9
@@ -274,10 +274,10 @@ public class Operation extends AbstractOperation
      */
     override public function cancel(id:String = null):AsyncToken
     {
-        if (showBusyCursor)
-        {
+        // if (showBusyCursor)
+        // {
 //            CursorManager.removeBusyCursor();
-        }
+        // }
         return super.cancel(id);
     }
 
@@ -289,10 +289,10 @@ public class Operation extends AbstractOperation
 
     override mx_internal function invoke(message:IMessage, token:AsyncToken = null):AsyncToken
     {
-        if (showBusyCursor)
-        {
+        // if (showBusyCursor)
+        // {
 //            CursorManager.setBusyCursor();
-        }
+        // }
 
         return super.invoke(message, token);
     }
@@ -302,10 +302,10 @@ public class Operation extends AbstractOperation
      */
     override mx_internal function preHandle(event:MessageEvent):AsyncToken
     {
-        if (showBusyCursor)
-        {
+        // if (showBusyCursor)
+        // {
 //            CursorManager.removeBusyCursor();
-        }
+        // }
 
         var wasLastCall:Boolean = activeCalls.wasLastCall(AsyncMessage(event.message).correlationId);
         var token:AsyncToken = super.preHandle(event);
diff --git a/frameworks/projects/MXRoyale/src/main/royale/mx/utils/ObjectUtil.as b/frameworks/projects/MXRoyale/src/main/royale/mx/utils/ObjectUtil.as
index 2ac930c..ef849ea 100644
--- a/frameworks/projects/MXRoyale/src/main/royale/mx/utils/ObjectUtil.as
+++ b/frameworks/projects/MXRoyale/src/main/royale/mx/utils/ObjectUtil.as
@@ -790,7 +790,7 @@ public class ObjectUtil
             // this test for checking whether an object is dynamic or not is 
             // pretty hacky, but it assumes that no-one actually has a 
             // property defined called "wootHackwoot"
-            object["wootHackwoot"];
+            var o:* = object["wootHackwoot"];
         }
         catch (e:Error)
         {