[Gnash-commit] gnash ChangeLog server/swf/tag_loaders.cpp [release_0_8_2_rc1]

[Sandro Santilli]

CVSROOT:        /sources/gnash
Module name:    gnash
Branch:         release_0_8_2_rc1
Changes by:     Sandro Santilli <strk>  08/02/25 15:25:42

Modified files:
        .              : ChangeLog 
        server/swf     : tag_loaders.cpp 

Log message:
        (sound_stream_head_loader): turn that annoying "No samples advertised"
        error to a debugging message.
        Whoever will take care of that will do it in head.

CVSWeb URLs:
http://cvs.savannah.gnu.org/viewcvs/gnash/ChangeLog?cvsroot=gnash&only_with_tag=release_0_8_2_rc1&r1=1.5711.2.22&r2=1.5711.2.23
http://cvs.savannah.gnu.org/viewcvs/gnash/server/swf/tag_loaders.cpp?cvsroot=gnash&only_with_tag=release_0_8_2_rc1&r1=1.186&r2=1.186.2.1

Patches:
Index: ChangeLog
===================================================================
RCS file: /sources/gnash/gnash/ChangeLog,v
retrieving revision 1.5711.2.22
retrieving revision 1.5711.2.23
diff -u -b -r1.5711.2.22 -r1.5711.2.23
--- ChangeLog   25 Feb 2008 11:13:05 -0000      1.5711.2.22
+++ ChangeLog   25 Feb 2008 15:25:41 -0000      1.5711.2.23
@@ -1,5 +1,8 @@
 2008-02-25 Sandro Santilli <address@hidden>
 
+       * server/swf/tag_loaders.cpp (sound_stream_head_loader): turn that
+         annoying "No samples advertised" error to a debugging message.
+         Whoever will take care of that will do it in head.
        * configure.ac: don't complain if high-level scripts for doc
          generations aren't available as long as lower-level ones are.
          Fixes bug reported in comment #12 of bug #22343.

Index: server/swf/tag_loaders.cpp
===================================================================
RCS file: /sources/gnash/gnash/server/swf/tag_loaders.cpp,v
retrieving revision 1.186
retrieving revision 1.186.2.1
diff -u -b -r1.186 -r1.186.2.1
--- server/swf/tag_loaders.cpp  19 Feb 2008 19:20:56 -0000      1.186
+++ server/swf/tag_loaders.cpp  25 Feb 2008 15:25:42 -0000      1.186.2.1
@@ -17,7 +17,7 @@
 // Foundation, Inc., 51 Franklin St, Fifth Floor, Boston, MA  02110-1301  USA
 //
 
-/* $Id: tag_loaders.cpp,v 1.186 2008/02/19 19:20:56 bwy Exp $ */
+/* $Id: tag_loaders.cpp,v 1.186.2.1 2008/02/25 15:25:42 strk Exp $ */
 
 #ifdef HAVE_CONFIG_H
 #include "gnashconfig.h"
@@ -1390,7 +1390,14 @@
     {
        // this seems common too, we'd need to reproduce with a custom
        // testcase to really tell if it's a problem or not...
-        log_error("No samples advertised for sound stream");
+       //
+       // The sample count, registered in a SoundInfo here, will
+       // be later fetched by SOUNDSTRAMBLOCK tag loader and passed
+       // over to sound_hanlder. Semantic of the sample_count there
+       // is not properly documented so dunno what would happen in
+       // case it will be zero (see sound_handler::fill_stream_data).
+       //
+        log_debug("No samples advertised for sound stream");
     }
 
     int latency = 0;