[Bug 758410] mpdparser: parsing of dateTime fields does not read the microseconds part

GStreamer (GNOME Bugzilla) bugzilla at gnome.org
Sat Nov 21 17:16:48 PST 2015


https://bugzilla.gnome.org/show_bug.cgi?id=758410

--- Comment #5 from Florin Apostol <florin.apostol at oregan.net> ---
(In reply to Thiago Sousa Santos from comment #3)
> Review of attachment 315979 [details] [review]:
> 
> ::: ext/dash/gstmpdparser.c
> @@ +902,1 @@
> +    seconds_as_double = second + (gdouble) microsecond / 1000000;
> 
> This assumes the field has all the 6 digits all the time. An entry of .123
> would need to be 123000 microseconds and not just 123.

thank you for spotting it. Somehow I missed it.
The new patch is simpler and should cover all cases. Please have a look at
that.

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