[gstreamer-bugs] [Bug 377280] [cdiocddasrc] issue if drive endianness != machine endianness

[GStreamer (bugzilla.gnome.org)]

Do not reply to this via email (we are currently unable to handle email
responses and they get discarded).  You can add comments to this bug at
http://bugzilla.gnome.org/show_bug.cgi?id=377280

  GStreamer | gst-plugins-good | Ver: 0.10.4





------- Comment #27 from Brian Cameron  2007-03-08 04:29 UTC -------

Yes, Wim, I don't think that this is the same endian-ness issue that you are
discussing.

This patch fixes a bug caused when machine endianness does not match drive
endianness.  All CD drives made today are little endian so the problem only
shows up when running on a big-endian machine using a little-endian CD drive. 

Some very old CD drives were big endian so I guess you'ld also see the problem
running on a little-endian machine with a very old drive.

I think it is okay to just not support older big-endian CD drives and tell
people they need to buy a new CD drive if they want to play CD audio.  CD
drives are not expensive these days.

Currently GStreamer does not work at all on Sparc for 99.9% of all users who
have the common little-endian drives.  I'd recommend applying this patch so
that CD audio playback is functional on Sparc.

That said, I would recommend making a minor modification to the patch so that
instead of using #ifdef SOLARIS you instead use 

#ifdef __BIG_ENDIAN__

so that this functionality works properly on other big endian machines, not
just Sparc.


-- 
Configure bugmail: http://bugzilla.gnome.org/userprefs.cgi?tab=email