[FFmpeg-devel] [PATCH 2/2] lavf/mov: strip com.apple.quicktime prefix in meta; parse creation date

Michael Niedermayer michael at niedermayer.cc
Wed Apr 19 04:54:36 EEST 2017


On Mon, Sep 12, 2016 at 08:24:44PM -0500, Rodger Combs wrote:
> 
> > On Sep 10, 2016, at 05:55, Michael Niedermayer <michael at niedermayer.cc> wrote:
> > 
> > On Fri, Sep 09, 2016 at 08:06:40PM -0500, Rodger Combs wrote:
> >> ---
> >> libavformat/mov.c | 16 ++++++++++++++++
> >> 1 file changed, 16 insertions(+)
> >> 
> >> diff --git a/libavformat/mov.c b/libavformat/mov.c
> >> index 54530e3..b75acd2 100644
> >> --- a/libavformat/mov.c
> >> +++ b/libavformat/mov.c
> >> @@ -42,6 +42,7 @@
> >> #include "libavutil/aes_ctr.h"
> >> #include "libavutil/sha.h"
> >> #include "libavutil/timecode.h"
> >> +#include "libavutil/parseutils.h"
> >> #include "libavcodec/ac3tab.h"
> >> #include "libavcodec/mpegaudiodecheader.h"
> >> #include "avformat.h"
> >> @@ -476,6 +477,21 @@ retry:
> >>                 return ret;
> >>             }
> >>             str[str_size] = 0;
> >> +            if (!strcmp(key, "com.apple.quicktime.creationdate")) {
> >> +                struct tm *ptm, tmbuf;
> >> +                int64_t timeval;
> >> +                if (av_parse_time(&timeval, str, 0) >= 0) {
> >> +                    time_t timet = timeval / 1000000;
> >> +                    if (ptm = gmtime_r(&timet, &tmbuf)) {
> >> +                        strftime(str, str_size, "%Y-%m-%d %H:%M:%S", ptm);
> >> +                        key = "creation_time";
> >> +                    }
> >> +                }
> >> +            } else if (!strcmp(key, "com.apple.quicktime.location.ISO6709")) {
> >> +                key = "location";
> >> +            } else if (!strncmp(key, "com.apple.quicktime.", 20)) {
> >> +                key += 20;
> >> +            }
> >>         }
> > 
> > this causes language specific entries no longer to be listed together
> > with their "parent" entries:
> 
> Huh; interesting. Not sure why this happens, but also not sure if it matters?

doesnt matter

btw this patch seems to have been forgotten

[...]
-- 
Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

You can kill me, but you cannot change the truth.
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 181 bytes
Desc: Digital signature
URL: <http://ffmpeg.org/pipermail/ffmpeg-devel/attachments/20170419/df3af607/attachment.sig>


More information about the ffmpeg-devel mailing list