[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