> The original change was to prefer ID3v2 tags over ID3v1. Programs such
> as
> iTunes would update the ID3v2 tag, but not the V1 tag, which was
> causing
> confusion to the user, as MP3::Info merges the two.
>
OK - I can see the issue. I don't see a particularly good
workaround for my case that doesn't involve some extra
scanning. For my id3v1 + ape files (and also id3v2 and mixed
id3v1+2), the patch below seems to be working for me. I'm
not sure how many folks use id3v1 + mp3gain (I moved to
id3v2 + mp3gain about a year ago) but the impact of this
change might be pretty significant.
Code:
--------------------
***************
*** 93,101 ****
# Also try and get an APE tag, which may have ReplayGain data.
my $tags = MP3::Info::get_mp3tag($fh, 2, undef, 1);
! if (!scalar keys %$tags) {
! $tags = MP3::Info::get_mp3tag($fh, 1);
}
# Now fetch the audio header information.
--- 93,101 ----
# Also try and get an APE tag, which may have ReplayGain data.
my $tags = MP3::Info::get_mp3tag($fh, 2, undef, 1);
! if (!scalar keys %$tags || !defined(MP3::Info::get_mp3tag($fh, 2))) {
! $tags = MP3::Info::get_mp3tag($fh, 1, undef, 1);
}
# Now fetch the audio header information.
--------------------
--
jth
------------------------------------------------------------------------
jth's Profile: http://forums.slimdevices.com/member.php?userid=48
View this thread: http://forums.slimdevices.com/showthread.php?t=23331
_______________________________________________
beta mailing list
[email protected]
http://lists.slimdevices.com/lists/listinfo/beta
