[taglib] lyrics support

[Brian Kerrick Nickel]

So you know, there is an ID3v2 tag which is identical (minus the identifier)
to the comments tag. If one were to copy/paste that the comment frame code,
there would be MP3 id3 support, and for APE/Vorbis/etc arbitrary names could
be chosen.

Just my two cents.

- Brian

On 6/12/06, Dominik Karall <dominik.karall at gmx.net> wrote:

> On Monday, 12. June 2006 23:13, Scott Wheeler wrote:
> > On Monday 12 June 2006 21:49, you wrote:
> > > hi wheeler!
> > >
> > > maybe you could add this patch to your taglib sources, this patch
> > > adds support for lyrics to the whole taglib library.
> > >
> > > ID3v1: no lyrics support possible
> > > ID3v2: support as provided in specs
> > > APE: added LYRICS tag (no lyrics support in specs, but a LYRICS
> > > tag doesn't affect others)
> > > XIPH: same as in APE
> > >
> > > thanks a lot!
> >
> > Unfortunately adding virtual methods to the base classes is not
> > binary compatible, so there are parts of the patch that I can
> > incorporate, but some of them will have to wait until the next
> > major version (2.0).
> >
> > Two other notes:
> >
> > - Generally it's preferable to send patches to the list rather than
> > my personal address so that other people involved in / using TagLib
> > can see and potentially review them
> >
> > - If you're generating patches, it's usually better to work with
> > the TagLib version from Subversion since that's more up to date.
> > There are instructions on the TagLib homepage for how to check out
> > the subversion sources.
>
> Hi Scott!
>
> Sorry, I wasn't sure if there is a mailinglist.
> I will update the patch for current svn and remove the virtual methods
> for the base class. Maybe I should divide the patch in separate diffs
> for each tag system..
> I will send a patch to the ML the next days!
>
> regards,
> dominik
> >
> >
-------------- next part --------------
An HTML attachment was scrubbed...
URL: http://mail.kde.org/pipermail/taglib-devel/attachments/20060614/5dedf47b/attachment.html