[Gnash-commit] gnash ChangeLog testsuite/libamf.all/test_objec...

[Sandro Santilli]

CVSROOT:        /sources/gnash
Module name:    gnash
Changes by:     Sandro Santilli <strk>  07/12/17 12:09:45

Modified files:
        .              : ChangeLog 
        testsuite/libamf.all: test_object.cpp 

Log message:
        ISO C++ forbids variable-size arrays (-pedantic).

CVSWeb URLs:
http://cvs.savannah.gnu.org/viewcvs/gnash/ChangeLog?cvsroot=gnash&r1=1.5192&r2=1.5193
http://cvs.savannah.gnu.org/viewcvs/gnash/testsuite/libamf.all/test_object.cpp?cvsroot=gnash&r1=1.10&r2=1.11

Patches:
Index: ChangeLog
===================================================================
RCS file: /sources/gnash/gnash/ChangeLog,v
retrieving revision 1.5192
retrieving revision 1.5193
diff -u -b -r1.5192 -r1.5193
--- ChangeLog   17 Dec 2007 11:12:09 -0000      1.5192
+++ ChangeLog   17 Dec 2007 12:09:44 -0000      1.5193
@@ -1,5 +1,10 @@
 2007-12-17 Sandro Santilli <address@hidden>
 
+       * testsuite/libamf.all/test_object.cpp: ISO C++ forbids variable-size
+         arrays (-pedantic).
+
+2007-12-17 Sandro Santilli <address@hidden>
+
        * testsuite/MovieTester.{cpp,h}: use a manual clock, to make
          time predictable. Expose an additional advanceClock method
          to control it (one way)
@@ -22,8 +27,8 @@
        * testsuite/misc-ming.all/new_child_in_unload_test.c, Makefile.am:
          new test for inspecting ::destroy(). We may want to destroy an 
character
          that is not unloaded.
-       * server/character.cpp: remove the assertion in charcter::destroy(), 
it's not
-         correct as tests confirmed.
+       * server/character.cpp: remove the assertion in charcter::destroy(),
+         it's not correct as tests confirmed.
          
 2007-12-16 Sandro Santilli <address@hidden>
 

Index: testsuite/libamf.all/test_object.cpp
===================================================================
RCS file: /sources/gnash/gnash/testsuite/libamf.all/test_object.cpp,v
retrieving revision 1.10
retrieving revision 1.11
diff -u -b -r1.10 -r1.11
--- testsuite/libamf.all/test_object.cpp        15 Dec 2007 06:58:41 -0000      
1.10
+++ testsuite/libamf.all/test_object.cpp        17 Dec 2007 12:09:45 -0000      
1.11
@@ -297,13 +297,14 @@
     } else {
         size_t s = 12;
         runtest.fail("RTMP Headers mismatch");
-        unsigned char hexint[(s*2)+1];
+        unsigned char* hexint = new unsigned char[(s*2)+1];
         hexify((unsigned char *)hexint, (unsigned char *)buf, s, true);
        hexint[s*2] = '\0';
         cerr << "buf is: 0x" << hexint << endl;
         hexify((unsigned char *)hexint, (unsigned char *)out, s, true);
        hexint[s*2] = '\0';
         cerr << "out is: 0x" << hexint << endl;
+       delete [] hexint;
     }
 
     tmpptr += rtmp.getHeaderSize();
@@ -483,7 +484,7 @@
     }    
 
     size_t hexsize = std::max(AMF_PACKET_SIZE, amf_obj.getTotalSize())*2;
-    unsigned char hexint[hexsize+1];
+    unsigned char* hexint = new unsigned char[hexsize+1];
     hexify((unsigned char *)hexint, (unsigned char *)buf, 
amf_obj.getTotalSize() + 10, true);
     hexint[hexsize] = '\0';
     cerr << "buf is: 0x" << hexint << ", size is: " << amf_obj.getTotalSize() 
<< endl;
@@ -491,6 +492,8 @@
     hexint[hexsize] = '\0';
     cerr << "out is: 0x" << hexint << ", size is: " << rtmp.getTotalSize() << 
endl;
     
+    delete [] hexint;
+    
 //    delete out;
 }