[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 #12 from rocky at panix.com  2006-11-22 00:04 UTC -------
Sorry it took so long - now I think I'm beginning to understand what you are
asking for.

If you look at any various CD-DA plugins, e.g vlc or xine you'll see that if
there is any byte swapping going on it's not happenning at the layer just above
CD read.  And all of those seem to work fine no matter what byte sex. In the
plugins I've done for those, cdio_read_audio_sectors() seems to be all that's
needed in those plugins. And a similar situation is true in the versions of the
plugins I didn't write.

The problem seems to come in when one uses CD paranoia. One issue here is that
it tries to *interpret* the data as 16-bit units such as to look for gaps of
silence. 

Personally of course, I would prefer if people use newer versions of libcdio.
So making that a *requirement* is, in my opinion, a good thing. It's not just
that there's more there in later versions, but various bugs and security flaws
have been fixed, and in some cases embarrasing gaps in my understanding of how
things work are corrected. I think most distributions provide version 0.76
which is over a year old; 0.78.2 is the current stable release.

Honestly, the only way I can think of to work around using a newer version
which has features you want is basically to write that code inside the
application. Overall I think you get both a harder-to-maintain and more complex
program; and the quality can also be less good. 

Finally as for the API - I'll make a note to document the behavior I mention
above after I verify that this is indeed what I write is the case.


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