[Gstreamer-bugs] [Bug 118473] New - mplex plugin does not work on Solaris

[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=118473

Changed by brian.cameron at ireland.sun.com.

--- shadow/118473	Mon Jul 28 05:53:53 2003
+++ shadow/118473.tmp.27590	Mon Jul 28 05:53:53 2003
@@ -0,0 +1,43 @@
+Bug#: 118473
+Product: GStreamer
+Version: HEAD CVS
+OS: SunOS
+OS Details: 
+Status: NEW   
+Resolution: 
+Severity: normal
+Priority: Normal
+Component: gst-plugins
+AssignedTo: gstreamer-maint at bugzilla.gnome.org                            
+ReportedBy: brian.cameron at ireland.sun.com               
+QAContact: gstreamer-maint at bugzilla.gnome.org
+TargetMilestone: HEAD
+URL: 
+Summary: mplex plugin does not work on Solaris
+
+The mplex plugin fails on Solaris due to a few problems.
+
++ Forte does not support __inline__.  It does support "inline".
+
++ multplex.cc alocates sector_buf with "sector_size".  The Forte
+  compiler does not allow you to allocate arrays to be the size
+  of a variable.  It would work with Forte if you allocate this
+  array with g_new and g_free.  If you want, you could use 
+  #ifdef __SUNPRO_C to tell when Forte is being used, so that the
+  current code could be retained for other compilers.
+
++ vector.cc fails to compile with this error:
+
+  "vector.cc", line 12: Error: Cannot assign Aunit* to Aunit**.
+
+  On this line:
+
+    buf = new (Aunit *)[AUStream::BUF_SIZE];
+
+  Note that in vector.hh, that "buf" is defined to be "Aunit **buf",
+  not an Aunit *, so something is clearly wrong here.  Perhaps this
+  error slips by gcc, but Forte noticed it.
+
+Two are addressed in the attached patch.  I am not sure how to best
+address the last issue mentioned, since my first shot was killed.
+See bug 118304.