[Bug 758384] dashdemux: tests: refactor into adaptive_engine components

GStreamer (GNOME Bugzilla) bugzilla at gnome.org
Thu Dec 10 03:27:45 PST 2015


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

--- Comment #23 from A Ashley <bugzilla at ashley-family.net> ---
(In reply to Thiago Sousa Santos from comment #21)
> Review of attachment 315952 [details] [review]:
> 
> ::: tests/check/elements/hlsdemux_m3u8.c
> @@ +1340,3 @@
> +  assert_equals_int64 (media->height, 352);
> +  assert_equals_int64 (media->bandwidth, 1251135);
> +  assert_equals_string (media->codecs, "\"avc1.42001f, mp4a.40.2\"");
> 
> Do we need to keep the ""? Are they always present or are they useful for
> some case?

I think they should be deleted, I was being lazy and avoiding touching code in
m3u8.c.

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