Bug 81002 - XMMS zz_mp3 minor fixup
XMMS zz_mp3 minor fixup
Product: Red Hat Public Beta
Classification: Retired
Component: xmms (Show other bugs)
All Linux
medium Severity low
: ---
: ---
Assigned To: Bill Nottingham
Depends On:
Blocks: 79579
  Show dependency treegraph
Reported: 2003-01-03 04:47 EST by Warren Togami
Modified: 2014-03-16 22:33 EDT (History)
3 users (show)

See Also:
Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of:
Last Closed: 2003-02-05 19:57:20 EST
Type: ---
Regression: ---
Mount Type: ---
Documentation: ---
Verified Versions:
Category: ---
oVirt Team: ---
RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: ---

Attachments (Terms of Use)
xmms-1.2.7-realmp3.patch (1.27 KB, patch)
2003-01-03 04:50 EST, Warren Togami
no flags Details | Diff

  None (edit)
Description Warren Togami 2003-01-03 04:47:53 EST
From Bugzilla Helper:
User-Agent: Mozilla/5.0 (X11; U; Linux i686; en-US; rv:1.2.1) Gecko/20021218

Description of problem:
XMMS currently has the zz_mp3 placeholder plugin that displays the patent error
message when people try to play MP3's.

When users install 3rd party MP3 plugins, this pop-up window continues to occur
while they successfully play MP3's.

These patches do the following:
* If one or more of the MP3 Input plugins are installed and enabled, zz_mp3 will
not pop up the warning window.
* It recognizes the difference between an enabled and disabled plugin and acts
* It recognizes libmpg123.so and libxmmsmad.so. If you are aware of other
possible MP3 input plugins for XMMS please e-mail me.

Version-Release number of selected component (if applicable):
Phoebe xmms-1.2.7-16.p
Comment 1 Warren Togami 2003-01-03 04:50:07 EST
Created attachment 89084 [details]

Patch against xmms/pluginenum.c.
Comment 2 Warren Togami 2003-01-03 04:52:00 EST
--- zz_mp3.c.orig       2002-12-31 02:50:10.000000000 -1000
+++ zz_mp3.c    2003-01-02 20:20:12.000000000 -1000
@@ -101,6 +101,7 @@
        guint16 wavid;
        if (!enabled) return FALSE;
+       if(real_mp3_plugin()) return FALSE;
        if (!strncasecmp(filename, "http://", 7))
        {                       /* We assume all http:// (except those ending in
.ogg) are mpeg -- why do we do that? */
Comment 3 Bill Nottingham 2003-01-03 12:36:32 EST
This shouldn't be needed; accroding to the xmms people, it should take the first
plugin alphabetically that accepts the current file type.
Comment 4 Warren Togami 2003-01-03 16:49:37 EST
According to my testing this isn't the case.  Quite often BOTH plugins would
"play" at the same time, playing the MP3 and popping up the Red Hat zz_mp3
window a the same time.  I added some printf()'s to see the order, and zz_mp3
was engaging before libmpg123.so in every case.
Comment 5 Warren Togami 2003-01-31 03:59:24 EST
Strange, in Phoebe2 only zz_mp3 would "play" while the installed MP3 plugin
doesn't.  Only when you disable zz_mp3 the installed MP3 plugin plays successfully.

Comment 6 Bill Nottingham 2003-02-05 19:57:20 EST
Fixed differently in 1.2.7-19.p. 
Comment 7 Warren Togami 2003-02-10 05:59:34 EST
Confirmed, your fix seems to work perfectly.

Note You need to log in before you can comment on or make changes to this bug.