[prev in list] [next in list] [prev in thread] [next in thread] 

List:       taglib-devel
Subject:    Re: itunes mp4 files
From:       Michael Pyne <pynm0001 () comcast ! net>
Date:       2006-01-13 22:53:45
Message-ID: 200601131801.39513.pynm0001 () comcast ! net
[Download RAW message or body]

[Attachment #2 (multipart/signed)]


On Monday 09 January 2006 17:21, Jochen Issing wrote:
> > The features are for now read only of almost all supported mp4 itunes
> > fields, cover is left out for now, but could be included as well. No
> > AudioProperties support yet, but will be my upcoming work. Write support
> > will be last step for implementation, because this will be the most
> > difficult part for me, especially rearding the offset tables, which I do
> > not need to touch now.

Hey, this is pretty awesome. :)  (Sorry for the late reply, btw)

> > Besides that the code is not completely cleaned up (some #if
> > 0-statements), because I might need those in future work. I tried to do
> > as much Doxygen comments as I was able to do - the License Header is
> > missing in some files as well.

May I ask how you're doing this work?  There is existing code out there to 
merge MPEG-4 into TagLib but it is derived from mplayer (or libfaad, I forget 
which) and is thus license incompatible.

If your work were to make its way to TagLib it would have to carry the same 
(or less restrictive) license.

But if you managed to write the code by reverse-engineering the format or 
using documentation (i.e. not just reimplementing from other code), that 
would probably be OK.

> > Please give me your opinion on the code, if the coding rules are matched
> > good enough, if the structure is Ok or whatever. At least, give it a try!
> > ;-)

Um, where *is* the code?

Regards,
 - Michael Pyne

[Attachment #5 (application/pgp-signature)]

_______________________________________________
taglib-devel mailing list
taglib-devel@kde.org
https://mail.kde.org/mailman/listinfo/taglib-devel


[prev in list] [next in list] [prev in thread] [next in thread] 

Configure | About | News | Add a list | Sponsored by KoreLogic