Re: [PATCH 6/6] test-ui: improved display of GDateTime values
- From: Guillaume Emont <guijemont igalia com>
- To: grilo-list gnome org
- Subject: Re: [PATCH 6/6] test-ui: improved display of GDateTime values
- Date: Thu, 30 Jun 2011 13:41:10 +0200
On 28/06/2011 11:28, Juan A. Suarez Romero wrote:
> On Wed, 2011-06-22 at 19:07 +0200, Guillaume Emont wrote:
>> return g_date_time_format (date_time, "GDateTime: %FT%H:%M:%S.%NZ")
>
> I'd remove the "GDateTime" string: we already know the date is of that
> type. We do not print any type for any key, so I don't see a reason to
> do in this case.
I suspect we sometime do, or to be exact, g_strdup_value_contents() can
do that sometimes.
Anyway, it was useful for debugging, to check that I had always a
GDateTime and not a gchar *, but it can be removed. Will do that.
>
> J.A.
>
>
> _______________________________________________
> grilo-list mailing list
> grilo-list gnome org
> http://mail.gnome.org/mailman/listinfo/grilo-list
>
[
Date Prev][Date Next] [
Thread Prev][
Thread Next]
[
Thread Index]
[
Date Index]
[
Author Index]