[Bug 783434] adaptivedemux: modify return value in each function

GStreamer (GNOME Bugzilla) bugzilla at gnome.org
Tue Jun 6 17:23:07 UTC 2017


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

--- Comment #3 from Jimmy Ohn <yongjin.ohn at lge.com> ---
(In reply to Sebastian Dröge (slomo) from comment #2)
> Comment on attachment 353189 [details] [review]
> adaptivedemux: modify return value in each function
> 
> Thanks for the patch, but why do you think this is an improvement?

we already declare ret variable above line. but it's not used in the each case
even though we can use it. I think that it's better to analyze modified source
code. Also, I refer to another demux code such qtdemux, avidemux.

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