[vlc-devel] [PATCH] access module for BlackMagic SDI cards

[Steinar H. Gunderson]

On Sat, Oct 02, 2010 at 06:51:50PM +0300, Rémi Denis-Courmont wrote:
>> New patch attached.
> 
> +    add_string( "decklink-video-connection", 0, NULL,
> +                 VIDEO_CONNECTION_TEXT, VIDEO_CONNECTION_LONGTEXT, true )
> 
> I guess there should be a predefined list of choices. Look for 
> change_integer_list or change_string_list.

Fixed.

I noticed during “vlc --module decklink” that it think all the options are
advanced. Is this right?

> +    psz_video_connection = var_CreateGetNonEmptyString( p_demux, "decklink-
> video-connection" );
> 
> You could use var_InheritString() here. It returns NULL if the value is 
> unspecified.

Changed.

/* Steinar */
-- 
Homepage: http://www.sesse.net/