[Webkit-unassigned] [Bug 116534] [GStreamer] Adjust internal size on http source element when receiving data if necessary
bugzilla-daemon at webkit.org
bugzilla-daemon at webkit.org
Thu Jun 6 06:08:49 PDT 2013
https://bugs.webkit.org/show_bug.cgi?id=116534
Carlos Garcia Campos <cgarcia at igalia.com> changed:
What |Removed |Added
----------------------------------------------------------------------------
Attachment #202424|commit-queue? |commit-queue-
Flag| |
--- Comment #5 from Carlos Garcia Campos <cgarcia at igalia.com> 2013-06-06 06:07:23 PST ---
(From update of attachment 202424)
View in context: https://bugs.webkit.org/attachment.cgi?id=202424&action=review
Please, fix coding style issues before landing.
> Source/WebCore/platform/graphics/gstreamer/WebKitWebSourceGStreamer.cpp:1019
> + // priv->size == 0 if received length on didReceiveResponse < 0
Nit: According to the coding style comments should finish with a period.
> Source/WebCore/platform/graphics/gstreamer/WebKitWebSourceGStreamer.cpp:1020
> + if (priv->size != 0 && priv->offset > priv->size) {
This is confusing offset and size are both unsigned so if offset == 0 it will never be > size. This could probably just be
if (priv->offset > priv->size)
> Source/WebCore/platform/graphics/gstreamer/WebKitWebSourceGStreamer.cpp:1021
> + GST_DEBUG_OBJECT(m_src, "Updating internal size from %ld to %ld", priv->size, priv->offset);
size and offset are guint64, you should probably use G_GUINT64_FORMAT instead of %ld.
--
Configure bugmail: https://bugs.webkit.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.
More information about the webkit-unassigned
mailing list