There are calls to gst_pad_link() that can be replaced by gst_element_link_pads_full() with GST_PAD_LINK_CHECK_NOTHING flag, preventing unnecessary checks and simplifying code. Also, getting rid of the gst_element_get_request_pad() calls will ease the migration to Gst 1.0.
Created attachment 179757 [details] patch A call to gst_pad_link() in GStreamerGWorld was left untouched because one of the pads is dynamic and is needed after linking; but gst_element_link_pads_full() doesn't give you a reference. Also, that code will not be ported to Gst 1.0 (as philn explained to me).
Created attachment 179759 [details] patch update to use '0' instead of 'NULL' previous patch incorrectly had 'NULL' instead of '0' (style checker didn't complain, hmm).
Comment on attachment 179759 [details] patch update to use '0' instead of 'NULL' Thanks!
Comment on attachment 179759 [details] patch update to use '0' instead of 'NULL' Clearing flags on attachment: 179759 Committed r137925: <http://trac.webkit.org/changeset/137925>
All reviewed patches have been landed. Closing bug.