Re: [PATCH 3/6] core: added grl_media_set_date_from_iso8601()

["Juan A. Suarez Romero" <jasuarez igalia com>]

On Wed, 2011-06-22 at 19:07 +0200, Guillaume Emont wrote:
>  
> +gboolean
> +grl_media_set_date_from_iso8601 (GrlMedia *media, const gchar *date)
> +{
> +  GTimeVal t = { 0, }; 


For me, the ideal solution would be GLib having a:

g_date_time_from_iso8601 (const gchar *iso_date, GDateTime *date_);

in the same spirit as g_time_val_from_iso8601(). And probably the other
way around: g_date_time_to_iso_8601().

As this is not true at this moment, I would add those functions in
grl-util, and making the user to use it to set a date from string.

Thus, if we add other date keys in the future, like _PUBLISHING_DATE, or
_RECORDING_DATE, or _MODIFIED_DATE, we do not need to add the
corresponding _from_iso8601() functions.


	J.A.