Re: [FFmpeg-devel] [PATCH] Adds support parsing the QuickTime Metadata Keys.

2015-10-26 Thread Tinglin Liu
On Mon, Oct 26, 2015 at 8:06 AM, Derek Buitenhuis < derek.buitenh...@gmail.com> wrote: > On 10/23/2015 7:41 PM, Tinglin Liu wrote: > > Derek, would you do the amend and push? Let me know if you need me to > > resend an amended patch. Thanks. > > Amende

Re: [FFmpeg-devel] [PATCH] Adds support parsing the QuickTime Metadata Keys.

2015-10-23 Thread Tinglin Liu
On Thu, Oct 22, 2015 at 4:57 PM, Derek Buitenhuis < derek.buitenh...@gmail.com> wrote: > On 10/23/2015 12:19 AM, wm4 wrote: > > Wrong, snprintf() always returns the number of characters the string > > would have been, even if the buffer size is smaller. > > That'll teach me to reply past

Re: [FFmpeg-devel] [PATCH] Adds support parsing the QuickTime Metadata Keys.

2015-10-22 Thread Tinglin Liu
'ilst'. Cheers, ​Tinglin​ On Thu, Oct 22, 2015 at 11:43 AM, Derek Buitenhuis < derek.buitenh...@gmail.com> wrote: > On 10/20/2015 7:29 PM, Tinglin Liu wrote: > > The Apple dev specification: > > > https://developer.apple.com/library/mac/documentation/QuickTime/QTF

[FFmpeg-devel] [PATCH] Adds support parsing the QuickTime Metadata Keys.

2015-10-22 Thread Tinglin Liu
The Apple dev specification: https://developer.apple.com/library/mac/documentation/QuickTime/QTFF/Metadata/Metadata.html Basically the structure is like: |--meta |hdlr |keys |ilst 1) The handler type in the metadata handler atom is ‘mdta’. 2) The key and value are stored separately

[FFmpeg-devel] [PATCH] Adds support parsing the QuickTime Metadata Keys.

2015-10-20 Thread Tinglin Liu
The Apple dev specification: https://developer.apple.com/library/mac/documentation/QuickTime/QTFF/Metadata/Metadata.html --- libavformat/isom.h | 3 +++ libavformat/mov.c | 77 +- 2 files changed, 74 insertions(+), 6 deletions(-) diff --git