search for: mp4_read_metadata

Hello I'd like to submit (if this has not been already done) a small patch which displays MP4 metadata. This peace of code has been taken from mp4info. Tell me what you think. Regards -------------- next part -------------- A non-text attachment was scrubbed... Name: in_mp4.c.diff Type: application/octet-stream Size: 2255 bytes Desc: not available Url :

...39;d like to submit (if this has not been already done) a small patch > > which displays MP4 metadata. This peace of code has been taken from > > mp4info. > > > > Tell me what you think. > > Looks pretty good. Could you make it a separate function (something > like mp4_read_metadata)? Also, I don't think there's any point in > reading anything but artist and title since those are the only fields > that get used. Oh, for consistency, call it Title instead of Name. > > I'll commit it with those changes. And thanks. Sorry I haven't had > more time...

...bedlam wrote: > Hello > I'd like to submit (if this has not been already done) a small patch > which displays MP4 metadata. This peace of code has been taken from > mp4info. > > Tell me what you think. Looks pretty good. Could you make it a separate function (something like mp4_read_metadata)? Also, I don't think there's any point in reading anything but artist and title since those are the only fields that get used. Oh, for consistency, call it Title instead of Name. I'll commit it with those changes. And thanks. Sorry I haven't had more time to review your other larg...