[Gnash-commit] gnash/server as_object.cpp

[Sandro Santilli]

CVSROOT:        /sources/gnash
Module name:    gnash
Changes by:     Sandro Santilli <strk>  07/06/02 08:56:22

Modified files:
        server         : as_object.cpp 

Log message:
        minor debugging errors cleanup

CVSWeb URLs:
http://cvs.savannah.gnu.org/viewcvs/gnash/server/as_object.cpp?cvsroot=gnash&r1=1.50&r2=1.51

Patches:
Index: as_object.cpp
===================================================================
RCS file: /sources/gnash/gnash/server/as_object.cpp,v
retrieving revision 1.50
retrieving revision 1.51
diff -u -b -r1.50 -r1.51
--- as_object.cpp       28 May 2007 15:41:05 -0000      1.50
+++ as_object.cpp       2 Jun 2007 08:56:22 -0000       1.51
@@ -119,6 +119,7 @@
        }
        catch (ActionException& exc)
        {
+               // TODO: check if this should be an 'as' error.. (log_aserror)
                log_error(_("Caught exception: %s"), exc.what());
                return false;
        }
@@ -224,9 +225,11 @@
        // SimpleProperty (if possible)
        if ( ! _members.setValue(key, val, *this) )
        {
-               log_error(_("Attempt to set read-only property ``%s''"
+               IF_VERBOSE_ASCODING_ERRORS(
+               log_aserror(_("Attempt to set read-only property ``%s''"
                        " on object ``%p''"),
                        key.c_str(), (void*)this);
+               );
        }
 
 }
@@ -248,7 +251,7 @@
                if ( ! _members.setValue(keylower, val, *this) )
                {
                        log_error(_("Attempt to initialize read-only property 
``%s''"
-                               " (%s) on object ``%p''"),
+                               " (%s) on object ``%p'' twice"),
                                keylower.c_str(), key.c_str(), (void*)this);
                        // We shouldn't attempt to initialize a member twice, 
should we ?
                        assert(0);
@@ -262,7 +265,7 @@
                if ( ! _members.setValue(key, val, *this) )
                {
                        log_error(_("Attempt to initialize read-only property 
``%s''"
-                               " on object ``%p''"),
+                               " on object ``%p'' twice"),
                                key.c_str(), (void*)this);
                        // We shouldn't attempt to initialize a member twice, 
should we ?
                        assert(0);
@@ -418,10 +421,12 @@
                        // set_member_flags will take care of case conversion
                        if ( ! set_member_flags(prop.c_str(), set_true, 
set_false) )
                        {
-                               log_error(_("Can't set propflags on object "
+                               IF_VERBOSE_ASCODING_ERRORS(
+                               log_aserror(_("Can't set propflags on object "
                                        "property %s "
                                        "(either not found or protected)"),
                                        prop.c_str());
+                               );
                        }
 
                        if ( next_comma == std::string::npos )