FaSTTUBe/GST-Tensordecoder-ov_ep
37
0
Fork 0
Code Issues Pull Requests 1 Actions Packages Projects Releases Wiki Activity

- properly ref/unref the caps while doing caps nego

Browse Source 
Original commit message from CVS:
- properly ref/unref the caps while doing caps nego
This commit is contained in:
Wim Taymans 2003-02-02 20:00:19 +00:00
parent 92749d3422
commit 869d913ff6
1 changed files with 29 additions and 12 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

41
ext/hermes/gstcolorspace.c
View File

@ -271,14 +271,19 @@ gst_colorspace_getcaps (GstPad *pad, GstCaps *caps)
GstColorspace *space; GstColorspace *space;
GstCaps *result; GstCaps *result;
GstCaps *peercaps; GstCaps *peercaps;
GstCaps *ourcaps; GstCaps *ourcaps, *temp;
space = GST_COLORSPACE (gst_pad_get_parent (pad)); space = GST_COLORSPACE (gst_pad_get_parent (pad));
/* we can do everything our peer can... */ /* we can do everything our peer can... */
peercaps = gst_caps_copy (gst_pad_get_allowed_caps (space->srcpad)); temp = gst_pad_get_allowed_caps (space->srcpad);
peercaps = gst_caps_copy (temp);
gst_caps_unref (temp);
/* and our own template of course */ /* and our own template of course */
ourcaps = gst_caps_copy (gst_pad_get_pad_template_caps (pad)); temp = gst_pad_get_pad_template_caps (pad);
ourcaps = gst_caps_copy (temp);
gst_caps_unref (temp);
/* merge them together, we prefer the peercaps first */ /* merge them together, we prefer the peercaps first */
result = gst_caps_prepend (ourcaps, peercaps); result = gst_caps_prepend (ourcaps, peercaps);
@ -303,14 +308,17 @@ gst_colorspace_sinkconnect (GstPad *pad, GstCaps *caps)
GST_INFO (GST_CAT_PROPERTIES, "size: %dx%d", space->width, space->height); GST_INFO (GST_CAT_PROPERTIES, "size: %dx%d", space->width, space->height);
space->sinkcaps = caps; gst_caps_replace_sink (&space->sinkcaps, caps);
peer = gst_pad_get_peer (pad); peer = gst_pad_get_peer (pad);
if (peer) { if (peer) {
if (gst_colorspace_srcconnect_func (pad, gst_pad_get_allowed_caps (space->srcpad), FALSE) < 1) { GstCaps *allowed = gst_pad_get_allowed_caps (space->srcpad);
if (gst_colorspace_srcconnect_func (pad, allowed, FALSE) < 1) {
space->sinkcaps = NULL; space->sinkcaps = NULL;
gst_caps_unref (allowed);
return GST_PAD_LINK_REFUSED; return GST_PAD_LINK_REFUSED;
} }
gst_caps_unref (allowed);
} }
return GST_PAD_LINK_OK; return GST_PAD_LINK_OK;
@ -327,7 +335,7 @@ gst_colorspace_srcconnect_func (GstPad *pad, GstCaps *caps, gboolean newcaps)
{ {
GstColorspace *space; GstColorspace *space;
GstCaps *peercaps; GstCaps *peercaps;
GstCaps *ourcaps; GstCaps *ourcaps, *to_intersect, *try_peercaps;
space = GST_COLORSPACE (gst_pad_get_parent (pad)); space = GST_COLORSPACE (gst_pad_get_parent (pad));
@ -348,20 +356,27 @@ gst_colorspace_srcconnect_func (GstPad *pad, GstCaps *caps, gboolean newcaps)
if (gst_pad_try_set_caps (space->srcpad, peercaps) > 0) { if (gst_pad_try_set_caps (space->srcpad, peercaps) > 0) {
space->type = GST_COLORSPACE_NONE; space->type = GST_COLORSPACE_NONE;
space->disabled = FALSE; space->disabled = FALSE;
gst_caps_unref (peercaps);
return GST_PAD_LINK_DONE; return GST_PAD_LINK_DONE;
} }
gst_caps_unref (peercaps);
} }
/* then see what the peer has that matches the size */
peercaps = gst_caps_intersect (caps, to_intersect = GST_CAPS_NEW (
GST_CAPS_NEW (
"colorspace_filter", "colorspace_filter",
"video/raw", "video/raw",
"width", GST_PROPS_INT (space->width), "width", GST_PROPS_INT (space->width),
"height", GST_PROPS_INT (space->height) "height", GST_PROPS_INT (space->height)
)); );
/* then see what the peer has that matches the size */
peercaps = gst_caps_intersect (caps, to_intersect);
gst_caps_unref (to_intersect);
/* we are looping over the caps, so we have to get rid of the lists */ /* we are looping over the caps, so we have to get rid of the lists */
peercaps = gst_caps_normalize (peercaps); try_peercaps = gst_caps_normalize (peercaps);
gst_caps_unref (peercaps);
peercaps = try_peercaps;
/* loop over all possibilities and select the first one we can convert and /* loop over all possibilities and select the first one we can convert and
* is accepted by the peer */ * is accepted by the peer */
@ -369,11 +384,13 @@ gst_colorspace_srcconnect_func (GstPad *pad, GstCaps *caps, gboolean newcaps)
if (colorspace_setup_converter (space, ourcaps, peercaps)) { if (colorspace_setup_converter (space, ourcaps, peercaps)) {
if (gst_pad_try_set_caps (space->srcpad, peercaps) > 0) { if (gst_pad_try_set_caps (space->srcpad, peercaps) > 0) {
space->disabled = FALSE; space->disabled = FALSE;
gst_caps_unref (try_peercaps);
return GST_PAD_LINK_DONE; return GST_PAD_LINK_DONE;
} }
} }
peercaps = peercaps->next; peercaps = peercaps->next;
} }
gst_caps_unref (try_peercaps);
/* we disable ourself here */ /* we disable ourself here */
space->disabled = TRUE; space->disabled = TRUE;
@ -549,7 +566,7 @@ gst_colorspace_change_state (GstElement *element)
space->converter = NULL; space->converter = NULL;
space->disabled = TRUE; space->disabled = TRUE;
space->type = GST_COLORSPACE_NONE; space->type = GST_COLORSPACE_NONE;
space->sinkcaps = NULL; gst_caps_replace (&space->sinkcaps, NULL);
break; break;
} }