[vlc-devel] [PATCH] Handle VLM events in libvlc

Sébastien Escudier sebastien-devel at celeos.eu
Tue May 19 11:55:39 CEST 2009



Hi

Please see included patch.
Some self comments about it :

@@ -63,6 +63,7 @@ struct libvlc_instance_t
 {
     libvlc_int_t *p_libvlc_int;
     vlm_t        *p_vlm;
+    libvlc_event_manager_t * p_event_manager;


-> I did not find a better place to put this event manager. I am open to
suggestions.



@@ -208,6 +214,33 @@ typedef struct libvlc_event_t
         {
             libvlc_time_t   new_length;
         } media_player_length_changed;
+
+        /* VLM media */
+        struct
+        {
+            const char * psz_media_name;
+        } vlm_media_added;
+
+        struct
+        {
+            const char * psz_media_name;
+        } vlm_media_removed;
[...]

-> As they all have the same variable (psz_media_name), should I create only one
structure and use it for all libvlc_VlmMedia* events ?

Thanks.
-------------- next part --------------
A non-text attachment was scrubbed...
Name: 0001-Handle-VLM-events-in-libvlc.patch
Type: text/x-patch
Size: 7650 bytes
Desc: not available
URL: <http://mailman.videolan.org/pipermail/vlc-devel/attachments/20090519/e68a592d/attachment.bin>


More information about the vlc-devel mailing list