After an evaluation, GNOME has moved from Bugzilla to GitLab. Learn more about GitLab.
No new issues can be reported in GNOME Bugzilla anymore.
To report an issue in a GNOME project, go to GNOME GitLab.
Do not go to GNOME Gitlab for: Bluefish, Doxygen, GnuCash, GStreamer, java-gnome, LDTP, NetworkManager, Tomboy.
Bug 643924 - [PATCH] youtube plugin not parsing date
[PATCH] youtube plugin not parsing date
Status: RESOLVED FIXED
Product: grilo
Classification: Other
Component: plugins
git master
Other Linux
: Normal normal
: ---
Assigned To: grilo-maint
grilo-maint
Depends on:
Blocks:
 
 
Reported: 2011-03-05 01:05 UTC by Michael Wood
Modified: 2011-03-07 12:54 UTC
See Also:
GNOME target: ---
GNOME version: ---

Attachments
Add initalise microseconds published value to 0 (917 bytes, patch)
2011-03-05 01:05 UTC, Michael Wood 		none Details | Review

Description Michael Wood 2011-03-05 01:05:20 UTC 
Created attachment 182531 [details] [review]
Add initalise microseconds published value to 0

I've been getting a lot of:

GLib-CRITICAL **: g_time_val_to_iso8601: assertion `time_->tv_usec >= 0 && time_->tv_usec < G_USEC_PER_SEC' failed

which seems to stem from not having set a value for tv_usec (the number of microseconds in the GTimeVal) given that we're never going to get (nor want) a more accurate time than seconds it should be safe to set this to 0 ?
Comment 1 Juan A. Suarez Romero 2011-03-07 12:54:57 UTC 
Pushed as

commit fb991cb13af37e77eb90d18687cec48abc7b64fc
Author: Michael Wood <michael.g.wood@linux.intel.com>
Date:   Sat Mar 5 00:57:56 2011 +0000

    youtube: Add initalise microseconds published value to 0
    
    This should mean that the GTimeValue can be correctly parsed to iso8601
    using g_time_val_to_iso8601
    
    Fixes https://bugzilla.gnome.org/show_bug.cgi?id=643924

 src/media/youtube/grl-youtube.c |    1 +
 1 files changed, 1 insertions(+), 0 deletions(-)