[Top][All Lists]

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

[Gnash-commit] [patch #8993] Patches to fix problems when `--disable-med

From: Nutchanon Wetchasit
Subject: [Gnash-commit] [patch #8993] Patches to fix problems when `--disable-media` configure option was used
Date: Tue, 10 May 2016 15:04:08 +0000 (UTC)
User-agent: Mozilla/5.0 (X11; Linux i686; rv:25.8) Gecko/20151123 Firefox/31.9 PaleMoon/25.8.1

Follow-up Comment #7, patch #8993 (project gnash):

>From the code causing link error, they are
SWF parser's *logging code*; when `log_parse()` tries to convert
`media::audioCodecType` to string, it invokes operator `ostream <<
media::audioCodecType`, which is only implemented in ``.

If I just strip them off using following patch and re-build (with patch 1 but
exclude MediaParser inlining), the build will complete without error.

diff --git a/libcore/swf/SoundStreamHeadTag.cpp
index 3b20909..65516a1 100644
--- a/libcore/swf/SoundStreamHeadTag.cpp
+++ b/libcore/swf/SoundStreamHeadTag.cpp
@@ -144,12 +144,6 @@ SoundStreamHeadTag::loader(SWFStream& in, TagType tag,
movie_definition& m,
         log_unimpl("SOUNDSTREAMHEAD contains %d unparsed bytes",
-        log_parse(_("sound stream head: format=%s, rate=%d, 16=%d, "
-            "stereo=%d, ct=%d, latency=%d"), format, streamSoundRate,
-            +streamSound16bit, +streamSoundStereo, +sampleCount, +latency);
-    );
     // Store all the data in a SoundInfo object
     const media::SoundInfo sinfo(format, streamSoundStereo,
                 streamSoundRate, sampleCount, streamSound16bit, latency);
diff --git a/libcore/swf/tag_loaders.cpp b/libcore/swf/tag_loaders.cpp
index c0da0ce..ed14a1e 100644
--- a/libcore/swf/tag_loaders.cpp
+++ b/libcore/swf/tag_loaders.cpp
@@ -186,12 +186,6 @@ define_sound_loader(SWFStream& in, TagType tag,
movie_definition& m,
         delaySeek = in.read_s16();
-        log_parse(_("define sound: ch=%d, format=%s, rate=%d, 16=%d, "
-            "stereo=%d, ct=%d, delay=%d"), id, format, sample_rate,
-            sample_16bit, stereo, sample_count, delaySeek);
-    );
     // If we have a sound_handler, ask it to init this sound.
     if (handler) {

Note: The video counterpart of this enum (media::videoCodecType
is also used in libcore's SWF parser at
but the code didn't try to print out the codec name, so it didn't cause any
more link error.

Gnash: 0.8.11dev (patched against git 484ca4c 29-Mar-2016)
System: Debian GNU/Linux 7.0 Wheezy i386


Reply to this item at:


  Message sent via/by Savannah

reply via email to

[Prev in Thread] Current Thread [Next in Thread]