[Bug 678031] gstdatetime: add parser and constructor for ISO 8601 datetime

GStreamer (bugzilla.gnome.org) bugzilla at gnome.org
Thu Jun 14 13:42:21 PDT 2012


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

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

           What    |Removed                     |Added
----------------------------------------------------------------------------
 Attachment #216458|none                        |needs-work
             status|                            |

--- Comment #4 from Tim-Philipp Müller <t.i.m at zen.co.uk> 2012-06-14 20:42:17 UTC ---
(From update of attachment 216458)
What I had in mind was slightly different, and should be even easier:

I think we should just have two functions, let's name them:

gst_date_time_new_from_iso8601_string(), and
gst_date_time_to_iso8601_string()

(or just _to_string() and new_from_string() - to be determined)

The _to_...string() function should serialise to the most compact form possible
IMHO, depending on which fields are valid. The DateTime knows which fields are
not set, so it can pick the right function itself instead of letting the caller
decide. So in effect _to_..string() should just be what string_check is now,
with a switch(datetime->fields) { ... }.

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


More information about the gstreamer-bugs mailing list