[gstreamer-bugs] [Bug 628408] Use GDateTime that has been released

[GStreamer (bugzilla.gnome.org)]

https://bugzilla.gnome.org/show_bug.cgi?id=628408
  GStreamer | gstreamer (core) | git

Tim-Philipp Müller <t.i.m> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |t.i.m at zen.co.uk

--- Comment #9 from Tim-Philipp Müller <t.i.m at zen.co.uk> 2010-10-12 12:30:23 UTC ---
> There are still 2 differences from our API to glib's.
> 
> 1) They use a double to represent seconds + microseconds. We have 2 separate
> fields in functions.

Makes sense to change our API to match GLib's here IMHO.  Don't think people
care too much about micro-seconds in our main usage scenario (tags).


> 2) They use a string for representing a timezone (+0300 or America/Recife or
> ...) as they can parse tzdata. And for the timezone offset they use
> microseconds resolution. We are using a double in hours resolution.
>   (...)
> For item 2 we could take our timezone handling to microseconds resolution too.
> glib does it because their datetime math API is based on microseconds, so it
> makes some sense. For us it doesn't make, but we get consistency with their
> API.
> 
> What do you think?

No strong opinion, I could live with both. I think passing timezone offsets in
microseconds is a bit weird though.

-- 
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.