[vlc-devel] [PATCH 01/12] bluray: Adding some event handling basics.

Jean-Baptiste Kempf jb at videolan.org
Mon Jan 23 01:23:29 CET 2012


On Sun, Jan 22, 2012 at 12:30:56AM +0100, Hugo Beauzée-Luyssen wrote :
> +    //Initialize the event queue, so we can receive then in Read later.

Please use consistent comments style.

> +static void     blurayHandleEvent( demux_t *p_demux, const BD_EVENT *e )
Weird spacing.

> +    switch ( e->event )

idem.

> +    {
> +    case    BD_EVENT_TITLE:
idem.

> +        break ;
idem.

Seems OK.

Best regards,


-- 
Jean-Baptiste Kempf
http://www.jbkempf.com/ - +33 672 704 734
Sent from my Electronic Device



More information about the vlc-devel mailing list