[Gnash-commit] gnash ChangeLog server/action.cpp server/swf/AS... [release_0_7_2]

[Sandro Santilli]

CVSROOT:        /sources/gnash
Module name:    gnash
Branch:         release_0_7_2
Changes by:     Sandro Santilli <strk>  06/11/07 11:47:21

Modified files:
        .              : ChangeLog 
        server         : action.cpp 
        server/swf     : ASHandlers.cpp 

Log message:
                * server/action.cpp, server/swf/ASHandlers.cpp: turn
                  more actionscript coding errors into warning and
                  wrap them in IF_VERBOSE_ASCODING_ERRORS.

CVSWeb URLs:
http://cvs.savannah.gnu.org/viewcvs/gnash/ChangeLog?cvsroot=gnash&only_with_tag=release_0_7_2&r1=1.1412.2.105&r2=1.1412.2.106
http://cvs.savannah.gnu.org/viewcvs/gnash/server/action.cpp?cvsroot=gnash&only_with_tag=release_0_7_2&r1=1.105.2.1&r2=1.105.2.2
http://cvs.savannah.gnu.org/viewcvs/gnash/server/swf/ASHandlers.cpp?cvsroot=gnash&only_with_tag=release_0_7_2&r1=1.84.2.4&r2=1.84.2.5

Patches:
Index: ChangeLog
===================================================================
RCS file: /sources/gnash/gnash/ChangeLog,v
retrieving revision 1.1412.2.105
retrieving revision 1.1412.2.106
diff -u -b -r1.1412.2.105 -r1.1412.2.106
--- ChangeLog   7 Nov 2006 11:43:00 -0000       1.1412.2.105
+++ ChangeLog   7 Nov 2006 11:47:21 -0000       1.1412.2.106
@@ -1,3 +1,9 @@
+2006-11-07 Sandro Santilli <address@hidden>
+
+       * server/action.cpp, server/swf/ASHandlers.cpp: turn
+         more actionscript coding errors into warning and
+         wrap them in IF_VERBOSE_ASCODING_ERRORS.
+
 2006-11-07 Tomas Groth Christensen <address@hidden>
 
        * backend/sound_handler_sdl.cpp: Made stop_all_sounds()

Index: server/action.cpp
===================================================================
RCS file: /sources/gnash/gnash/server/action.cpp,v
retrieving revision 1.105.2.1
retrieving revision 1.105.2.2
diff -u -b -r1.105.2.1 -r1.105.2.2
--- server/action.cpp   30 Oct 2006 14:28:48 -0000      1.105.2.1
+++ server/action.cpp   7 Nov 2006 11:47:21 -0000       1.105.2.2
@@ -244,10 +244,12 @@
        }
        else
        {
-               log_error(
+                       IF_VERBOSE_ASCODING_ERRORS(
+               log_warning(
                        "error in call_method(): "
                        "'%s' is neither a C nor an ActionScript function\n",
                        method.to_string());
+                       );
        }
 
        return val;

Index: server/swf/ASHandlers.cpp
===================================================================
RCS file: /sources/gnash/gnash/server/swf/ASHandlers.cpp,v
retrieving revision 1.84.2.4
retrieving revision 1.84.2.5
diff -u -b -r1.84.2.4 -r1.84.2.5
--- server/swf/ASHandlers.cpp   5 Nov 2006 20:09:06 -0000       1.84.2.4
+++ server/swf/ASHandlers.cpp   7 Nov 2006 11:47:21 -0000       1.84.2.5
@@ -16,7 +16,7 @@
 
 //
 
-/* $Id: ASHandlers.cpp,v 1.84.2.4 2006/11/05 20:09:06 strk Exp $ */
+/* $Id: ASHandlers.cpp,v 1.84.2.5 2006/11/07 11:47:21 strk Exp $ */
 
 #ifdef HAVE_CONFIG_H
 #include "config.h"
@@ -2002,8 +2002,10 @@
                if (function.get_type() != as_value::AS_FUNCTION &&
                    function.get_type() != as_value::C_FUNCTION)
                {
-                   log_error("error in call_function: '%s' is not a function",
-                             function_name.c_str());
+                       IF_VERBOSE_ASCODING_ERRORS(
+                   log_warning("error in call_function: '%s' is not a "
+                               "function", function_name.c_str());
+                       );
                }
        }
        else
@@ -2542,25 +2544,40 @@
 
     }
 
-    if (!obj) {
-        log_error("call_method invoked in something that "
+    if (!obj)
+    {
+        IF_VERBOSE_ASCODING_ERRORS(
+          log_warning("call_method invoked in something that "
                   "doesn't cast to an as_object: %s",
                   obj_value.to_string());
-    } else {
+        );
+    }
+    else
+    {
         as_value method;
-        if (obj->get_member(method_name, &method)) {
+        if (obj->get_member(method_name, &method))
+        {
             if (method.get_type() != as_value::AS_FUNCTION &&
-                method.get_type() != as_value::C_FUNCTION) {
-                log_error("call_method: '%s' is not a method",
+              method.get_type() != as_value::C_FUNCTION)
+          {
+              IF_VERBOSE_ASCODING_ERRORS(
+                log_warning("call_method: '%s' is not a method",
                           method_name.c_str());
-            } else {
+              );
+          }
+          else
+          {
                 result = call_method( method, &env, obj, nargs,
                                       env.get_top_index() - 3);
             }
-        } else {
-            log_error("call_method can't find method %s "
+        }
+        else
+        {
+            IF_VERBOSE_ASCODING_ERRORS(
+                log_warning("call_method can't find method %s "
                       "for object %s (%p)", method_name.c_str(), 
                       typeid(*obj).name(), (void*)obj);
+            );
         }
     }