[Gnash-commit] gnash ChangeLog server/as_value.cpp

[Sandro Santilli]

CVSROOT:        /sources/gnash
Module name:    gnash
Changes by:     Sandro Santilli <strk>  07/10/03 16:27:25

Modified files:
        .              : ChangeLog 
        server         : as_value.cpp 

Log message:
                * server/as_value.cpp (to_bool*): assert that OBJECT and 
FUNCTION
                  values are never null.

CVSWeb URLs:
http://cvs.savannah.gnu.org/viewcvs/gnash/ChangeLog?cvsroot=gnash&r1=1.4511&r2=1.4512
http://cvs.savannah.gnu.org/viewcvs/gnash/server/as_value.cpp?cvsroot=gnash&r1=1.80&r2=1.81

Patches:
Index: ChangeLog
===================================================================
RCS file: /sources/gnash/gnash/ChangeLog,v
retrieving revision 1.4511
retrieving revision 1.4512
diff -u -b -r1.4511 -r1.4512
--- ChangeLog   3 Oct 2007 14:58:30 -0000       1.4511
+++ ChangeLog   3 Oct 2007 16:27:24 -0000       1.4512
@@ -1,5 +1,10 @@
 2007-10-03 Sandro Santilli <address@hidden>
 
+       * server/as_value.cpp (to_bool*): assert that OBJECT and FUNCTION
+         values are never null.
+
+2007-10-03 Sandro Santilli <address@hidden>
+
        * server/as_value.{cpp,h}: stop using m_string_value as a cache
          for convertion to string (propedeutic for storing the string
          into the union). clean m_string_value inside drop_refs

Index: server/as_value.cpp
===================================================================
RCS file: /sources/gnash/gnash/server/as_value.cpp,v
retrieving revision 1.80
retrieving revision 1.81
diff -u -b -r1.80 -r1.81
--- server/as_value.cpp 3 Oct 2007 14:58:31 -0000       1.80
+++ server/as_value.cpp 3 Oct 2007 16:27:25 -0000       1.81
@@ -350,8 +350,8 @@
                        return this->m_boolean_value;
                case OBJECT:
                case AS_FUNCTION:
-                       // it is possible we'll need to convert to number 
anyway first
-                       return m_object_value != NULL;
+                       assert(m_object_value != NULL);
+                       return true;
                case MOVIECLIP:
                        return true;
                default:
@@ -385,8 +385,8 @@
                        return this->m_boolean_value;
                case OBJECT:
                case AS_FUNCTION:
-                       // it is possible we'll need to convert to number 
anyway first
-                       return m_object_value != NULL;
+                       assert(m_object_value != NULL);
+                       return true;
                case MOVIECLIP:
                        return true;
                default:
@@ -420,8 +420,8 @@
                        return this->m_boolean_value;
                case OBJECT:
                case AS_FUNCTION:
-                       // it is possible we'll need to convert to number 
anyway first
-                       return m_object_value != NULL;
+                       assert(m_object_value != NULL);
+                       return true;
                case MOVIECLIP:
                        return true;
                default: