[Bug 658309] souphttpsrc: Add support for getting content size via content-range header

GStreamer (bugzilla.gnome.org) bugzilla at gnome.org
Mon Dec 30 05:51:44 PST 2013


https://bugzilla.gnome.org/show_bug.cgi?id=658309
  GStreamer | gst-plugins-good | unspecified

--- Comment #20 from Steve Maynard <steve at secondstryke.com> 2013-12-30 13:51:41 UTC ---
(In reply to comment #19)
> Review of attachment 264845 [details]:
> 
> ::: ext/soup/gstsouphttpsrc.c
> @@ +792,3 @@
>            gst_message_new_duration_changed (GST_OBJECT (src)));
>      }
> +  } else {
> 
> Why is this still having the content size block above? Can't we merge that into
> gst_soup_http_src_determine_size()?

Maybe I misunderstood the previous comment "Can't we do the if-branch above in
here too?". Are you asking for the code from line #777 through #794 be moved
into the the function gst_soup_http_src_determine_size?

-- 
Configure bugmail: https://bugzilla.gnome.org/userprefs.cgi?tab=email
------- 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