elzaksspro / gstreamer-java

Automatically exported from code.google.com/p/gstreamer-java
0 stars 0 forks source link

add GstPadAPI.gst_pad_add_event_probe(), Pad.addEventProbe() #20

Closed GoogleCodeExporter closed 8 years ago

GoogleCodeExporter commented 8 years ago
It would be nice to have this function, as not all events are trappable by
using Element.connect() nor Bus.connect(). Included is a (tested) patch
against the latest hg repository. Thanks.

Original issue reported on code.google.com by tsha...@gmail.com on 25 Jul 2008 at 8:58

Attachments:

GoogleCodeExporter commented 8 years ago
Thanks, I'll put that in later.

It probably needs gst_pad_remove_probe too ?

Original comment by wmeiss...@gmail.com on 2 Aug 2008 at 12:33

GoogleCodeExporter commented 8 years ago
You are right, of course: here is a patch with gst_pad_remove_probe. I have 
added a
Map<Pad.EVENT_PROBER,Integer> to the Pad class to keep track of added event 
probers
so they may be removed later. 

I have updated PadTest.addEventProbe() accordingly to also test 
gst_pad_remove_probe()

Original comment by tsha...@gmail.com on 12 Aug 2008 at 12:09

Attachments:

GoogleCodeExporter commented 8 years ago
I had this half implemented, but not checked in yet, so its slightly different 
to your patch, but it should be done 
now.

Original comment by wmeiss...@gmail.com on 12 Aug 2008 at 4:17

GoogleCodeExporter commented 8 years ago
Marking as fixed

Original comment by wmeiss...@gmail.com on 1 Sep 2008 at 11:51