[Webkit-unassigned] [Bug 29717] [GTK] missing support for anamorphic PAR video size

bugzilla-daemon at webkit.org bugzilla-daemon at webkit.org
Thu Oct 1 23:54:32 PDT 2009


https://bugs.webkit.org/show_bug.cgi?id=29717





--- Comment #8 from Philippe Normand <pnormand at igalia.com>  2009-10-01 23:54:32 PDT ---
(In reply to comment #6)
> (From update of attachment 40435 [details])
> > @@ -178,14 +172,13 @@ webkit_video_sink_set_caps(GstBaseSink* bsink, GstCaps* caps)
> >      priv->width = width;
> >      priv->height = height;
> >  
> > -    /* We dont yet use fps or pixel aspect into but handy to have */
> > -    priv->fps_n = gst_value_get_fraction_numerator(fps);
> > -    priv->fps_d = gst_value_get_fraction_denominator(fps);
> > +    /* We dont yet use fps but handy to have */
> > +    if (!gst_structure_get_fraction(structure, "framerate",
> > +                                    &priv->fps_n, &priv->fps_d))
> > +        return FALSE;
> 
> So, I fail to see why we would return here, because failing to retrieve
> something we do not even use failed. Does this failing mean we will, say, fail
> to play the video? Returning here will only cause the pixel aspect ratio to not
> be set, in case we would be able to fetch it, but not this, so maybe just set a
> sane default, like you are doing for pixel aspect ratio.

I only kept the previous behavior of the code in this patch...
What would be a "sane default" fps? :)

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