[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Libcdio-devel] Paranoia and CD-Extra
From: |
R. Bernstein |
Subject: |
Re: [Libcdio-devel] Paranoia and CD-Extra |
Date: |
Thu, 27 Sep 2007 20:41:38 -0400 |
Patric Guimond's patch has finally been committed to CVS. Sorry again
for the delay. Alas cutting and pasting from the posting patch
cleanly on the current CVS sources, so I had to intervene. Please look
over to verify I haven't botched things.
Also, I made some small changes to the FixupTOC patch. Some C
compilers require that declarations not follow statements unless you
start a new block. In particular it looked like this was happening to
variable last_ses_lsn. And I don't like magic constants like 75 put in
code. (More could be done here since there is a #define for the number
of frames in a 2-second gap. But I guess I'm getting old and less
rigorous about such stuff.)
Patrick Guimond writes:
> Hi
>
> I already sent a patch about this. It went through unnoticed though... :/
>
> You can find it here:
> http://lists.gnu.org/archive/html/libcdio-devel/2006-08/msg00000.html
>
> Patrick Guimond
>
>
> David Stockwell wrote:
> > Ripping a CD in CD-Extra format (with links to a website following the
> > last track, I find that cdio-paranoia continues to read the track beyond
> > the end of the track until it hits the "Extra" portion, recording
> > everything thereafter as a "skip". This appears to be a known problem
> > with Paranoia.
> >
> > FWIW, I am using cdio_paranoia_read_limited, with a callback to report
> > the status, so I do know what is going on. In reading the track, I am
> > seeking the first LSN of the track, then looping through the track from
> > the first LSN to the last, similar to the following:
> >
> > for (lsn_t lsnCurr=lsnFirst; lsnCurr<=lsnLast; ++lsnCurr) {
> > pParaBuffer = cdio_paranoia_read_limited(pParanoia, cbParanoia, 20);
> > }
> >
> > So, why am I reading past the end of the last track?
> >
> > Looking at the includes, I see a call to
> > cdio_paranoia_set_range(pParanoia, {beginning LSN}, {ending LSN}) which
> > is not really documented, but appears to set limits on how far paranoia
> > will read.
> >
> > Including that call in my source, I get a good compile, but the link
> > fails with "undefined reference to cdio_paranoia_set_range".
> >
> > I have installed 0.78 into /usr/local (the basic distro includes 0.77 in
> > /usr).
> >
> > Finally, this is the only disk I cannot read/rip, so I believe it has to
> > do with strangeness from this format.
> >
> > Any help?
> >
> > David Stockwell
> >
> >
> > ------------------------------------------------------------------------
> >
> > _______________________________________________
> > Libcdio-devel mailing list
> > address@hidden
> > http://lists.gnu.org/mailman/listinfo/libcdio-devel
>
>
>
> _______________________________________________
> Libcdio-devel mailing list
> address@hidden
> http://lists.gnu.org/mailman/listinfo/libcdio-devel
>