quietdragon wrote: > Robin, > > I tried sending you an email. No reply. Perhaps it got junked.
I may have got it and forgot to reply - sorry! > > I'm using v0.3.0rc1 from your web site. > > > - The script uses "--preset standard" to run the lame encoder. I'd > like to use "-V4 --new-vbr", but prefer not to hack it into the > source. > > Would you entertain a new "--lame" switch which would allow the > preferred lame encoding settings to be obtained from the command > line? I would. Can you file an enhancement request on my tracker? http://robinbowes.com/projects/flac2mp3/newticket > > (I've attached a diff that includes this flag.) > - I notice that you bundle MP3/Tag. Looking around I see that this > appears to be a fairly old version. I wonder if there are latent bugs > or mis-features related to the next two issues. Possibly - I'll look into upgrading. > - I find my UTF-8 tags get scrambled. I'm pretty sure the cause is > the utf8toLatin1 procedure. I've replaced it with Encode::decode() > and force UTF-8 as the output character coding scheme. With this > change, my UTF-8 tags stay intact. I must admit - my knowledge is thin on the ground in this area. I don't think I handle character encoding right at all. Is this in your patch? Can you provide a patch against latest trunk code (or 0.30rc1 - not much has changed) ? Please create a ticket (see above) and attach the patch. > - I couldn't get the MD5 checksum test to work. After probing I saw > the MP3::Tag appears to return the TXXX tag as "MD5 / n" in the > Description! Huh? I made a corresponding change in the source code > but perhaps the problem is actually elsewhere. Strange. Works for me. What exactly do you mean? > The attached diff contains changes to address the above for you to > peruse. I'll take a look as soon as I get chance, but it's best to raise stuff like this by creating tickets, otherwise I forget! Thanks, R. _______________________________________________ ripping mailing list [email protected] http://lists.slimdevices.com/lists/listinfo/ripping
