[Bug 707975] qtdemux: Can't handle datetimes before 1970 yet

Tue Sep 24 15:27:30 PDT 2013

https://bugzilla.gnome.org/show_bug.cgi?id=707975
  GStreamer | gst-plugins-good | unspecified

Thiago Sousa Santos <thiago.sousa.santos> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|NEW                         |RESOLVED
         Resolution|                            |FIXED
   Target Milestone|HEAD                        |1.2.x

--- Comment #3 from Thiago Sousa Santos <thiago.sousa.santos at collabora.co.uk> 2013-09-24 22:27:23 UTC ---
commit dc02d91c14847e48b6e830ff58f2dd27ecae336d
Author: Thiago Santos <thiago.sousa.santos at collabora.com>
Date:   Mon Sep 16 11:20:51 2013 -0300

    qtdemux: add code to parse creation time earlier than 1970

    Use g_date_time seconds manipulation to allow to cover the quicktime
    spec for creation_time. It uses seconds since 1904.

    Both paths could be done using the generic approach of seconds since
    1904 with GDateTime handling, but the first path using seconds from
    1970 should be more commonly found and avoids a few objects creation and
    ref/unref, so keep it there for performance.

    Additionally, the code for handling seconds since 1970 changed from >
    to >= because having 0 seconds since 1970 is also a valid case for that
    path to handle.

    https://bugzilla.gnome.org/show_bug.cgi?id=707975

-- 
Configure bugmail: https://bugzilla.gnome.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the QA contact for the bug.
You are the assignee for the bug.