Bug 81002 - XMMS zz_mp3 minor fixup
Summary: XMMS zz_mp3 minor fixup
Keywords:
Status: CLOSED RAWHIDE
Alias: None
Product: Red Hat Public Beta
Classification: Retired
Component: xmms
Version: phoebe
Hardware: All
OS: Linux
medium
low
Target Milestone: ---
Assignee: Bill Nottingham
QA Contact:
URL:
Whiteboard:
Depends On:
Blocks: 79579
TreeView+ depends on / blocked
 
Reported: 2003-01-03 09:47 UTC by Warren Togami
Modified: 2014-03-17 02:33 UTC (History)
3 users (show)

Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Clone Of:
Environment:
Last Closed: 2003-02-06 00:57:20 UTC


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

Description Warren Togami 2003-01-03 09:47:53 UTC
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.

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

Fix
===
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
accordingly.
* 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 09:50:07 UTC
Created attachment 89084 [details]
xmms-1.2.7-realmp3.patch

Patch against xmms/pluginenum.c.

Comment 2 Warren Togami 2003-01-03 09:52:00 UTC
--- 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 17:36:32 UTC
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 21:49:37 UTC
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 08:59:24 UTC
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-06 00:57:20 UTC
Fixed differently in 1.2.7-19.p. 

Comment 7 Warren Togami 2003-02-10 10:59:34 UTC
Confirmed, your fix seems to work perfectly.



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