[Gnash-commit] /srv/bzr/gnash/rtmp r9640: makeNumber properties correctly.

[rob]

------------------------------------------------------------
revno: 9640
committer: address@hidden
branch nick: rtmp
timestamp: Wed 2008-09-17 22:05:29 -0600
message:
  makeNumber properties correctly.
modified:
  libamf/element.cpp
  libamf/element.h
  testsuite/libcore.all/AsValueTest.cpp

=== modified file 'libamf/element.cpp'
--- a/libamf/element.cpp        2008-09-18 03:41:34 +0000
+++ b/libamf/element.cpp        2008-09-18 04:05:29 +0000
@@ -545,8 +545,8 @@
         setName(name);
     }
     _type = Element::NUMBER_AMF0;
-//    check_buffer(AMF0_NUMBER_SIZE);
-//     _buffer->copy(num);
+    check_buffer(AMF0_NUMBER_SIZE);
+    _buffer->copy(num);
     return *this;
 }
 

=== modified file 'libamf/element.h'
--- a/libamf/element.h  2008-09-18 03:41:34 +0000
+++ b/libamf/element.h  2008-09-18 04:05:29 +0000
@@ -119,8 +119,6 @@
     Element &operator=(const std::string &str);
     Element &operator=(bool flag);
     
-    // These create the other "special" AMF types.
-
     // A Null String is a string with a length of zero
     Element &makeNullString();
 
@@ -150,7 +148,7 @@
     Element &makeNull();
     Element &makeNull(const std::string &name);
 
-    // Tterminate an object
+    // Terminate an object
     Element &makeObjectEnd();
 
     // Make an AMF Object. This is an AMF type that support having properties.

=== modified file 'testsuite/libcore.all/AsValueTest.cpp'
--- a/testsuite/libcore.all/AsValueTest.cpp     2008-09-15 06:39:48 +0000
+++ b/testsuite/libcore.all/AsValueTest.cpp     2008-09-18 04:05:29 +0000
@@ -237,7 +237,7 @@
     std::auto_ptr<Element> el1 = as1.to_element();
     Element *fooel = el1->getProperty(0);
     Element *barel = el1->getProperty(1);
-    if ((el1->getType() == Element::OBJECT_AMF0)
+    if ((el1.get()->getType() == Element::OBJECT_AMF0)
         && (fooel->getType() == Element::STRING_AMF0)
         && (strcmp(fooel->getName(), "foo") == 0)
         && (strcmp(fooel->to_string(), "bar") == 0)