Eric Lesage | 11 Dec 2009 09:31
Picon

[patch] cdparanoia sample offset inconsistency

Hello,

Apologies if this is a duplicate. I've sent a previous version to the list 
but it appears it did not get through. The version of the patch included 
herein is better anyway.

I've encountered a problem when using the -O (--sample-offset) option. 
Currently, the code checks whether the specified offset is more than 588 
samples, and if so biases the sectors in the (internal) TOC so that the 
offset becomes less than 588.

However, this creates an inconsistency when the span is analyzed: the span 
might both refer to (corrected) values from the TOC (e.g. end of disk) and 
uncorrected values (e.g. absolute sector positions).

E.g.: [.2000]- (read from sector 2000 to end of disk).

The included patch fixes this by adding the offset in sectors specified 
using absolute positions.

Best regards,

--

-- 
Eric Lesage
Index: interface/cdda_interface.h
===================================================================
--- interface/cdda_interface.h	(revision 16745)
+++ interface/cdda_interface.h	(working copy)
 <at>  <at>  -66,6 +66,8  <at>  <at> 
(Continue reading)


Gmane