On Sun, Dec 09, 2018 at 08:53:35PM +0100, Thomas Schmitt wrote: > Hi, > > > I've just finished a bug-finding session on cd-paranoia. > > What repo are you working with ? > I'm now looking at > https://github.com/rocky/libcdio-paranoia/tree/master
I've just made a fork so I can keep track of my changes. Here's my changes, with your comments applied: https://github.com/vext01/libcdio-paranoia/commit/9d2cf9f1d3573a7d3d38c245f558b9b0f34afb9d There's also one other fix in here for when the user specifies a span: https://github.com/vext01/libcdio-paranoia/commit/9d2cf9f1d3573a7d3d38c245f558b9b0f34afb9d#diff-2dcfa705a400429e0003ed129ecefb76L151 (There's a lot of mixed tabs and spaces in the code. I'll leave it up to you guys if you want to fix that.) The result works, and the tests pass. I guess we should do a test on Linux and NetBSD before we start thinking about getting my branches in shape for merging. Are you able to do the Linux testing? -- Best Regards Edd Barrett http://www.theunixzoo.co.uk