[Bug 751766] New: m3u8: test_playlist_with_doubles_duration failure

[GStreamer (GNOME Bugzilla)]

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

            Bug ID: 751766
           Summary: m3u8: test_playlist_with_doubles_duration failure
    Classification: Platform
           Product: GStreamer
           Version: unspecified
                OS: Linux
            Status: NEW
          Severity: normal
          Priority: Normal
         Component: gst-plugins-bad
          Assignee: gstreamer-bugs at lists.freedesktop.org
          Reporter: vineeth.tm at samsung.com
        QA Contact: gstreamer-bugs at lists.freedesktop.org
     GNOME version: ---

test_playlist_with_doubles_duration test of hlsdemux_m3u8 fails because,
when a string is converted to double using g_ascii_strtod and the same value is
then converted to GstClockTime, for some values of float, the conversion is not
happening properly

The failure is as below.
elements/hlsdemux_m3u8.c:561:F:m3u8client:test_playlist_with_doubles_duration:0:
'file->duration' (10320999999) is not equal to '10.321 * (1000000 *
(__extension__ (1000LL)))' (10321000000)

Let us take the example of 10.321 float value, given in DOUBLES_PLAYLIST of
hlsdemux_m3u. This when got using g_ascii_strtod and then multiplying by
GST_SECOND,
instead of giving 10321000000, the value is coming as 10320999999
For 10.322 also there is error in calculations.
for 10.323, the value is coming properly as 10323000000. For 10.320 also it is
proper.

Similarly for 9.92, there is error, along with 9.93, 9.94, 9.95 are giving
errors, but 9.90, 9.91, 9.96, 9.97 are giving proper values

Not sure what exactly is happening and how does the calculations vary based on
different values.
But if i change the duration variable to gdouble, then there is no error.
And while passing the duration value to gst_m3u8_media_file_new, which expects
GstClockTime duration, then it gets converted properly and all the calculations
are happening properly without any failures.

-- 
You are receiving this mail because:
You are the QA Contact for the bug.
You are the assignee for the bug.