[Gstreamer-bugs] [Bug 100582] Changed - wavparse plugin doesn't support big endian machines

[bugzilla-daemon at widget.gnome.org]

Please do not reply to this email- if you want to comment on the bug, go to the
URL shown below and enter your comments there.

http://bugzilla.gnome.org/show_bug.cgi?id=100582

Changed by uraeus at linuxrising.org.

--- shadow/100582	Sun Dec  8 16:29:21 2002
+++ shadow/100582.tmp.4405	Thu Dec 12 06:43:55 2002
@@ -1,18 +1,18 @@
 Bug#: 100582
 Product: GStreamer
 Version: 0.4.2
 OS: Linux
 OS Details: Gentoo Linux/PPC
-Status: REOPENED   
-Resolution: 
+Status: RESOLVED   
+Resolution: FIXED
 Severity: major
 Priority: Normal
 Component: gst-plugins
 AssignedTo: gstreamer-maint at bugzilla.gnome.org                            
-ReportedBy: nall at themountaingoats.net               
+ReportedBy: nall at gentoo.org               
 QAContact: gstreamer-maint at bugzilla.gnome.org
 TargetMilestone: 0.4.0
 URL: 
 Summary: wavparse plugin doesn't support big endian machines
 
 the wavparse plugin in the gst-plugins component doesn't work on big endian
@@ -25,18 +25,18 @@
 ------- Additional Comments From Uraeus at linuxrising.org  2002-12-07 11:12 -------
 A patch would be even better than more info :). Anyway we have some
 PPC developers on GStreamer so hopefully I can get them to look into
 this if 
 you can't make us a patch.
 
-------- Additional Comments From nall at themountaingoats.net  2002-12-07 14:12 -------
+------- Additional Comments From nall at gentoo.org  2002-12-07 14:12 -------
 Created an attachment (id=12823)
 patch to correctly parse wav headers for big endian machines
 
 
-------- Additional Comments From nall at themountaingoats.net  2002-12-07 14:12 -------
+------- Additional Comments From nall at gentoo.org  2002-12-07 14:12 -------
 a patch you want, a patch you got!
 
 i used the define __BIG_ENDIAN__ to qualify stuff. i know this works
 on gcc-3.2.1. i'm not sure about others (figuring that out in
 configure is probably the correct solution). 
 
@@ -44,32 +44,35 @@
 
 i've also "fixed" the "signed" parameter in caps, however it looks
 like osssink doesn't support unsigned data.
 
 enjoy.
 
-------- Additional Comments From nall at themountaingoats.net  2002-12-07 15:15 -------
+------- Additional Comments From nall at gentoo.org  2002-12-07 15:15 -------
 Created an attachment (id=12825)
 changed to use G_BIG_ENDIAN and fixed unsigned data problem
 
 
-------- Additional Comments From nall at themountaingoats.net  2002-12-07 15:17 -------
+------- Additional Comments From nall at gentoo.org  2002-12-07 15:17 -------
 Created an attachment (id=12826)
 an 8-bit unsigned wav i used for testing
 
 
-------- Additional Comments From nall at themountaingoats.net  2002-12-07 16:22 -------
+------- Additional Comments From nall at gentoo.org  2002-12-07 16:22 -------
 Created an attachment (id=12828)
 fix to use G*_FROM_LE
 
 
 ------- Additional Comments From Uraeus at linuxrising.org  2002-12-08 07:29 -------
 Ok Jon, patch applied and commited to CVS and you added to the credits
 section in the release notes. Thanks :)
 
-------- Additional Comments From nall at themountaingoats.net  2002-12-08 16:28 -------
+------- Additional Comments From nall at gentoo.org  2002-12-08 16:28 -------
 Created an attachment (id=12854)
 there's also bad code in gst-libs/gst/riff/riffutils.c. this fixes that file.
 
 
-------- Additional Comments From nall at themountaingoats.net  2002-12-08 16:29 -------
+------- Additional Comments From nall at gentoo.org  2002-12-08 16:29 -------
 reopening due to addition patch that needs to be applied.
+
+------- Additional Comments From Uraeus at linuxrising.org  2002-12-12 06:43 -------
+ok, this last patch also commited, thanks Jon