Re: [Libcdio-devel] cd-text not working for bin/cue driver?

[Rocky Bernstein]

On Wed, Oct 17, 2012 at 7:16 PM, Robert William Fuller <
address@hidden> wrote:

> Also, there does not seem to be a unit test for getting cd-text from
> bin/cue which is perhaps why this was missed?
>
> There IS a unit test for getting cd-text from nrg (Nero) files.


I guess what you are referring to is code test/driver/nrg.c  So ok,
test/driver.bincue.c should be improved.

I'll look into that when I get a chance. Or perhaps you can create a test
that currently fails that should succeed when the code is fixed.

 I had to look at old emails to refresh my memory but apparently I wrote
> that code as well as the original support for cd-text from Nero (as an aid
> to my memory, this should be added to my spot in the THANKS file: "various
> fixes to Nero driver and cd-text for Nero?")
>

That part is easy enough to fix - in git now. If there is other stuff or
you want the wording different, just go in and change it in git. I think
you should be able to commit to git.


>
> Rob
>
>
> On 10/17/2012 06:53 PM, Robert William Fuller wrote:
>
>> As promised, I have started testing with libcdio from git. I am seeing
>> what might be some problems with the new cd-text support. Specifically,
>> I am reading a bin/cue image and it does not seem to be getting the
>> cd-text fields from the cue file. It works with 0.83, but not with git.
>>
>> I will include my code and the cue file. The code is part of my
>> cd-ripper. It prints out the cd-text fields as part of creating a new
>> cue sheet. Maybe I am doing something wrong with the new interface.
>> Here's my code:
>>
>> void cdio2_fprint_cd_text(FILE *cueFile, CdIo_t *cdObj, track_t track,
>> const char *prefix)
>> {
>> int i, quoted;
>> cdtext_t *cdtext;
>> const char *field;
>>
>> #ifdef CUED_HAVE_CDTEXT_V2
>> cdtext = cdio_get_cdtext(cdObj);
>> #else
>> cdtext = cdio_get_cdtext(cdObj, track);
>> #endif
>>
>> if (cdtext) {
>> for (i = MIN_CDTEXT_FIELD; i < MAX_CDTEXT_FIELDS; ++i) {
>> #ifdef CUED_HAVE_CDTEXT_V2
>> field = cdtext_get_const(cdtext, (cdtext_field_t) i, track);
>> #else
>> field = cdtext_get_const((cdtext_**field_t) i, cdtext);
>> #endif
>> // checking for field[0] is for Nero, which sometime has zero length
>> if (field && field[0]) {
>> quoted = strchr(field, ' ') ? 1 : 0;
>> fprintf(cueFile, "%s%s %s%s%s\n", prefix,
>> cdtext_field2str((cdtext_**field_t) i),
>> quoted ? "\"" : "", field, quoted ? "\"" : "");
>> }
>> }
>> }
>> }
>>
>> Here is the cue file that was read by the libcdio bin/cue driver:
>>
>> REM DISCID 8E09C30B
>> REM COMMENT "ExactAudioCopy v0.95b4"
>> PERFORMER "The Band"
>> TITLE "Music From Big Pink"
>> FILE "The Band - Music From Big Pink.wav" WAVE
>> TRACK 01 AUDIO
>> TITLE "Tears Of Rage"
>> PERFORMER "The Band"
>> INDEX 01 00:00:00
>> TRACK 02 AUDIO
>> TITLE "To Kingdome Come"
>> PERFORMER "The Band"
>> INDEX 00 05:17:18
>> INDEX 01 05:21:19
>> TRACK 03 AUDIO
>> TITLE "In A Station"
>> PERFORMER "The Band"
>> INDEX 00 08:37:34
>> INDEX 01 08:42:16
>> TRACK 04 AUDIO
>> TITLE "Caledonia Mission"
>> PERFORMER "The Band"
>> INDEX 00 12:09:39
>> INDEX 01 12:14:59
>> TRACK 05 AUDIO
>> TITLE "The Weight"
>> PERFORMER "The Band"
>> INDEX 00 15:07:33
>> INDEX 01 15:12:40
>> TRACK 06 AUDIO
>> TITLE "We Can Talk"
>> PERFORMER "The Band"
>> INDEX 00 19:43:60
>> INDEX 01 19:48:38
>> TRACK 07 AUDIO
>> TITLE "Long Black Veil"
>> PERFORMER "The Band"
>> INDEX 00 22:49:12
>> INDEX 01 22:53:34
>> TRACK 08 AUDIO
>> TITLE "Chest Fever"
>> PERFORMER "The Band"
>> INDEX 00 25:52:20
>> INDEX 01 25:57:08
>> TRACK 09 AUDIO
>> TITLE "Lonesome Suzie"
>> PERFORMER "The Band"
>> INDEX 00 31:07:51
>> INDEX 01 31:12:37
>> TRACK 10 AUDIO
>> TITLE "This Wheel's On Fire"
>> PERFORMER "The Band"
>> INDEX 00 35:10:43
>> INDEX 01 35:14:40
>> TRACK 11 AUDIO
>> TITLE "I Shall Be Released"
>> PERFORMER "The Band"
>> INDEX 00 38:24:66
>> INDEX 01 38:27:22
>>
>
>
>