[Gstreamer-bugs] [Bug 94289] Changed - cdparanoia plugin refusing to change state when cd is not in drive

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

Changed by ds at schleef.org.

--- shadow/94289	Mon Sep 30 03:32:57 2002
+++ shadow/94289.tmp.22072	Mon Jul 28 23:39:19 2003
@@ -1,9 +1,9 @@
 Bug#: 94289
 Product: GStreamer
-Version: cvs
+Version: HEAD CVS
 OS: other
 OS Details: 
 Status: ASSIGNED   
 Resolution: 
 Severity: normal
 Priority: Normal
@@ -25,6 +25,37 @@
 workaround for now, but it's just a lot nicer to be able to change cds
 without having to worry about states and stuff.
 
 ------- Additional Comments From jorn at nl.linux.org  2002-09-26 11:47 -------
 Actually, we cannot lock the tray because when stuff crashes the tray
 wont open anymore ;)
+
+------- Additional Comments From ds at schleef.org  2003-07-28 23:39 -------
+As I see it, cdparanoia should do the following under an "error
+condition", i.e., CD being ejected:
+
+ - PLAYING:
+
+  send an EOS event and throw a 'media ejected' signal (or possibly error)
+
+ - PAUSED or READY:
+
+  throw a media ejected signal
+
+ - NULL
+
+  do nothing
+
+If the tray is open, and the element is in the PAUSED or READY state,
+the element is in an inconsistent state, and any attempts to change
+the state READY->PAUSED or PAUSED->PLAYING should throw an error. 
+Changes toward NULL are ok.
+
+This does not handle 1) closing the tray when a play button is
+pressed, or 2) automatically playing when a disk is inserted.  The
+former *could* be done during the NULL->READY state transition,
+depending on a parameter, but I don't recommend it.  I would prefer to
+see a separate interface to control the availability of the CD tray
+resource.
+
+I'd like to hear comments from other gstreamer developers on the
+media-ejected signal vs. error issue.