[prev in list] [next in list] [prev in thread] [next in thread] 

List:       alsaplayer-devel
Subject:    [Alsaplayer-devel](no subject)
From:       szaszg () hu ! inter ! net
Date:       2004-03-04 8:13:28
Message-ID: E1Aynyy-0002Zn-00 () smart ! eusc ! inter ! net
[Download RAW message or body]

Hi Andy and others!

There are some bug in the cdda input plugin. The most #$@#%@#:

    - when you use 'alsaplayer CD.cdda' than the CD.cdda track expanded,
ant the original CD.cdda 'track'
    (with 0:00 playtime) still exist in the playlist. so, if you
    start it again, alsaplayer (the cdda plugin) expand it again...
    If you use playlist looping, your playlist grow and grow and grow...
    
    - the 1024 char device name and the strcpy (segmentation fault)
    
    - can't choose CD-ROM device *runtime* (remote client + 2 or more
    CD-ROM... ;-)
    

So, here is a patch:

the plugin now accept URI like addressing:
    cdda://[tracks][:discID][@device]

where tracks:
    - nothing => mean all track from the CD
    - track number
    - range (e.g. 3-7)
    - any comma separated combinations of above
    
    e.g. cdda://1-3,7,2-4 => alsaplayer play track 1,2,3,7,2,3,4 :-)

discID:
    - 0 (zero) => plugin play tracks from any CD
    - a cddb disc id => plugin play tracks *only* this CD
    /* - '?' alsaplayer remember to current CD ... *not working yet* */

device:
    CD-ROM device path...

so, e.g. cdda://1-3,7:0xd8112c0f@/dev/cdroms/cdrom2 =>
 alsaplayer play track 1,2,3,7 from CD-ROM on /dev/cdroms/cdrom2 if
 the inserted CD cddb disc id equal with 0xd8112c0f :-)
 
plugin not expand multiple tracks, so in the above example make only
one item on the playlist...
 (very difficult, to avoid the growing playlist effect, because the
  plugin can't 'frozen' the playlist or change the item name.)

===============
There are three compatibility level with the old cdda plugin

- if #defined CDDA_ONLY_URI :
    plugin accept only cdda://... names
- if #undef CDDA_ONLY_URI :
    plugin accept the old naming scheme, but CD.cdda not expanded
- if #undef CDDA_ONLY_URI and #define CDDA_FULL_COMPAT :
    plugin accept the old naming scheme, and expand CD.cdda, 
    *and* you enjoy with the growing playlist effect :-)

Please try it, and use it :-)

Gergely



_______________________________________________
alsaplayer-devel mailing list
alsaplayer-devel@lists.tartarus.org
http://lists.tartarus.org/mailman/listinfo/alsaplayer-devel
[prev in list] [next in list] [prev in thread] [next in thread]